a70541a685b5836ed7db79b17fdfe799e0037d70: Bug 977991 patch 6 - Use the faster eRestyle_StyleAttribute path for style attribute changes. r=birtles
L. David Baron <dbaron@dbaron.org> - Sat, 13 Sep 2014 06:17:37 -0700 - rev 205213
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Bug 977991 patch 6 - Use the faster eRestyle_StyleAttribute path for style attribute changes. r=birtles
7726bc49a0160256546320f2a7d9252daf1e43ae: Bug 977991 patch 5 - Add eRestyle_ChangeAnimationPhase to switch between the with-animation and without-animation styles. r=birtles
L. David Baron <dbaron@dbaron.org> - Sat, 13 Sep 2014 06:17:37 -0700 - rev 205212
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Bug 977991 patch 5 - Add eRestyle_ChangeAnimationPhase to switch between the with-animation and without-animation styles. r=birtles This is an additional bit on nsRestyleHint that says that the restyling operation should also perform all the work needed to switch between style-without-animation and style-with-animation (based on nsPresContext::IsProcessingAnimationStyleChange). These concepts will go away in bug 960465. Note that we don't want this behavior for the animation-only style update code (bug 996796, etc.), and I wanted to make this explicit so that it was clear when it was happening, and so that it was clear what code should be removed when we git rid of it. This is the workaround needed to land bug 977991 prior to bug 960465. (I think there's also a minor dependency in the other direction, so we need a workaround one way or the other.) Note that this depends on bug 1057231.
6c6d274614cba5ec60e719fb43aadf22e04c7f8c: Bug 977991 patch 4 - Ensure we don't try to use the style attribute optimization on pseudo-elements. r=birtles
L. David Baron <dbaron@dbaron.org> - Sat, 13 Sep 2014 06:17:36 -0700 - rev 205211
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Bug 977991 patch 4 - Ensure we don't try to use the style attribute optimization on pseudo-elements. r=birtles This is needed because patch 1 and patch 3 did not add a mechanism that allows restyling pseudo-elements, which would be substantially more work and very little use (since the only case in which they have style attributes is for our internal use on the ::-moz-color-swatch pseudo-element). RestyleUndisplayedChildren does not need the same fix because it's only used on elements, and never on pseudo-elements or anonymous boxes.
c570f81a5dcc8f834e3c5cee19c013b98b3fcf8b: Bug 977991 patch 3 - Add ability for RuleNodeWithReplacement to replace the style attribute rule and its important rule. r=birtles
L. David Baron <dbaron@dbaron.org> - Sat, 13 Sep 2014 06:17:36 -0700 - rev 205210
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Bug 977991 patch 3 - Add ability for RuleNodeWithReplacement to replace the style attribute rule and its important rule. r=birtles This allows posting a restyle that says that only the rule(s) from the StyleAttrSheet cascade level will be replaced, which avoids running selector matching. Part 4 will ensure that we only invoke this code for element styles (and not pseudo-element or anonymous box styles). Despite that, I prefer having the runtime check here as well given that it's a very simple way to ensure we don't do something silly that might have security implications.
04139f81b4d124b062038f9d6b362b43f732bbca: Bug 977991 patch 2 - Add mCheckForImportantRules to the information nsStyleSet::RuleNodeWithReplacement keeps about cascade levels. r=birtles
L. David Baron <dbaron@dbaron.org> - Sat, 13 Sep 2014 06:17:36 -0700 - rev 205209
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Bug 977991 patch 2 - Add mCheckForImportantRules to the information nsStyleSet::RuleNodeWithReplacement keeps about cascade levels. r=birtles This is needed to support doing replacements of levels that can contain important rules, which will happen in patch 3.
c81eb3a1336b2ab5f4e83b8b5df394dfea8be37e: Bug 977991 patch 1 - Expose variant of RulesMatching on nsHTMLCSSStyleSheet that is less work to call. r=birtles
L. David Baron <dbaron@dbaron.org> - Sat, 13 Sep 2014 06:17:36 -0700 - rev 205208
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Bug 977991 patch 1 - Expose variant of RulesMatching on nsHTMLCSSStyleSheet that is less work to call. r=birtles This allows nsStyleSet::RuleNodeWithReplacement to call it without constructing an entire (and unnecessary) ElementRuleProcessorData, which will happen in patch 3.
2105fa333c57b06e4f24a20bb16be0fa9c9aea1b: Bug 977991 patch 0 - Add bitwise operators to nsRestyleHint. r=birtles
L. David Baron <dbaron@dbaron.org> - Sat, 13 Sep 2014 06:17:36 -0700 - rev 205207
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Bug 977991 patch 0 - Add bitwise operators to nsRestyleHint. r=birtles This avoids having to cast back to nsRestyleHint after using bitwise operators, and allows |= (etc.). (In the future we should consider converting nsRestyleHint, and probably also nsChangeHint, to use MOZ_MAKE_ENUM_CLASS_BITWISE_OPERATORS.)
b4ed770bf60ac603ec2bd025dd047f0f34641362: Bug 1058346 patch 2 - Add eRestyle_SVGAttrAnimations and support it in nsStyleSet::RuleNodeWithReplacement. r=birtles
L. David Baron <dbaron@dbaron.org> - Sat, 13 Sep 2014 06:17:35 -0700 - rev 205206
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Bug 1058346 patch 2 - Add eRestyle_SVGAttrAnimations and support it in nsStyleSet::RuleNodeWithReplacement. r=birtles This allows posting a restyle that says that only the rule(s) from the SVGAttrAnimationSheet cascade level will be replaced, which avoids running selector matching. This is needed to land bug 977991 prior to landing bug 960465, since that requires replacing all levels that contain animations. (I'll rename this to match the name determined in bug 1057231.)
4cc74bf25e33ef5488e18b2529cbbd18c2347a55: Bug 1058346 patch 1 - Expose a simpler variant of RulesMatching from SVGAttrAnimationRuleProcessor. r=birtles
L. David Baron <dbaron@dbaron.org> - Sat, 13 Sep 2014 06:17:35 -0700 - rev 205205
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Bug 1058346 patch 1 - Expose a simpler variant of RulesMatching from SVGAttrAnimationRuleProcessor. r=birtles This allows nsStyleSet::RuleNodeWithReplacement to call it without constructing an entire (and unnecessary) ElementRuleProcessorData, which will happen in patch 2.
1e975131a9ecf05cf7ae56908597c46f6d2644d4: Bug 1057231 patch 4 - Move SVG Attribute Animation rules from the pres hint level to their own cascade level. r=dholbert
L. David Baron <dbaron@dbaron.org> - Sat, 13 Sep 2014 06:17:35 -0700 - rev 205204
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Bug 1057231 patch 4 - Move SVG Attribute Animation rules from the pres hint level to their own cascade level. r=dholbert This is only a very slight reordering of their position in the cascade, since they were previously walked at the end of nsSVGElement::WalkContentStyleRules, which was called near the end of nsHTMLStyleSheet::RulesMatching. So the only change should be that they now take priority over the xml:lang rule added by nsHTMLStyleSheet, a rule with which they do not interact.
2d2ccca02fda856eeb27937513d0fc83c0215423: Bug 1057231 patch 3 - Add new cascade level for rules from SMIL Animation of SVG attributes to the style set. r=dholbert
L. David Baron <dbaron@dbaron.org> - Sat, 13 Sep 2014 06:17:35 -0700 - rev 205203
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Bug 1057231 patch 3 - Add new cascade level for rules from SMIL Animation of SVG attributes to the style set. r=dholbert I will fix the indentation of gCascadeLevels in bug 977991 patch 2, which adds a member to each element of the array. Note that this bumps the maximum sheetType from 8 to 9 (and number of them from 9 to 10), which does not require updating NS_RULE_NODE_LEVEL_MASK, since NS_RULE_NODE_LEVEL_MASK currently has 4 bits and allows a maximum of 15.
8e402720eb00746ad1d74c18d0beb0adc8174432: Bug 1057231 patch 2 - Add SVG Attribute Animation rule processor to the document. r=dholbert
L. David Baron <dbaron@dbaron.org> - Sat, 13 Sep 2014 06:17:35 -0700 - rev 205202
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Bug 1057231 patch 2 - Add SVG Attribute Animation rule processor to the document. r=dholbert
c8f0bae4450434f40b2f27488b91fbc99fdd8708: Bug 1057231 patch 1 - Add a rule processor class for rules from SMIL Animation of SVG attributes. r=dholbert
L. David Baron <dbaron@dbaron.org> - Sat, 13 Sep 2014 06:17:34 -0700 - rev 205201
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Bug 1057231 patch 1 - Add a rule processor class for rules from SMIL Animation of SVG attributes. r=dholbert This patch series (to move SVG Animation rules to their own level of the cascade) is needed so that they can participate in the mechanism for updating only animation styles to the present time without touching any other styles. In theory, this should have been done as part of 996796 or even as part of prior work on off main thread animations, except that it doesn't seem possible to have SVG Animations and CSS Transitions interacting on the same element and have that element's style out of date, which means it wasn't necessary for the existing code to update only animation styles. However, bug 960465 will use this similar mechanism as the new mechanism for making sure transitions start in a reasonable way (replacing the mechanism of phases, going from style without animation to style with animation via nsPresContext::IsProcessingAnimationStyleChange). This will require that, for SVG Animations to continue interacting correctly with CSS Transitions, that they have their own cascade level so that they can participate in RestyleManager::UpdateOnlyAnimationStyles. Additionally, this is needed even sooner, for bug 977991, which needs (temporarily, until bug 960465 lands) a mechanism for updating the style attribute style *and* all animation-phase-dependent styles. Yes, it's a little bit annoying to have to have another class, and another object per-document for this, but that's currently what comes with being a level of the cascade. But it's one class and one object per document, and I believe there will be substantial benefits to having all rule processor levels be only-animation-related or not-animation-related (except for the style attribute level, which is simple enough that it isn't needed yet).
310f2eaded12133a621bddb3a249eb52fb71a6ca: Bug 1066911 - Allow eRestyle_Force{,Descendants} in RuleNodeWithReplacement. r=heycam
L. David Baron <dbaron@dbaron.org> - Sat, 13 Sep 2014 06:17:34 -0700 - rev 205200
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Bug 1066911 - Allow eRestyle_Force{,Descendants} in RuleNodeWithReplacement. r=heycam This is fine since the assertion is just checking that we're not calling RuleNodeWithReplacement in cases when stronger restyling (i.e., restyling that does more work) is needed; the new eRestyle_Force and eRestyle_ForceDescendants flags control whether we restyle at all. This probably should have been done in bug 931668 (since bug 996796 landed before bug 931668, although not before the patches were written), but it doesn't actually show up in our test suite until bug 977991 (or maybe slightly earlier in bug 1057231 or bug 1058346).
cd918760da393a9e7472a13e0f1c6e965d04af13: Merge b-i to m-c, a=merge
Phil Ringnalda <philringnalda@gmail.com> - Sun, 14 Sep 2014 11:38:37 -0700 - rev 205199
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Merge b-i to m-c, a=merge
c22fbd74e303b5ce1d2fe38da0993d694aee9e36: Bumping manifests a=b2g-bump
B2G Bumper Bot <release+b2gbumper@mozilla.com> - Sat, 13 Sep 2014 11:17:01 -0700 - rev 205198
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Bumping manifests a=b2g-bump
a47e73f662b1a5aa31041bf767b4efd2d8700bb1: Bumping gaia.json for 2 gaia revision(s) a=gaia-bump
B2G Bumper Bot <release+b2gbumper@mozilla.com> - Sat, 13 Sep 2014 11:10:47 -0700 - rev 205197
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Bumping gaia.json for 2 gaia revision(s) a=gaia-bump ======== https://hg.mozilla.org/integration/gaia-central/rev/37ff07d95f18 Author: Kevin Grandon <kevingrandon@yahoo.com> Desc: Merge pull request #23975 from KevinGrandon/bug_1066339_statusbar_height_variables Bug 1066339 - Use CSS variables for statusbar height ======== https://hg.mozilla.org/integration/gaia-central/rev/4118d6feef43 Author: Kevin Grandon <kevingrandon@yahoo.com> Desc: Bug 1066339 - Use CSS variables for statusbar height r=alive
c1ca0e74d3cbc0c844b6f7822047985cba77b10c: Merge m-c to b-i
Phil Ringnalda <philringnalda@gmail.com> - Sat, 13 Sep 2014 10:16:41 -0700 - rev 205196
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Merge m-c to b-i
362fc6985227719fc243325e84fa716ee52477b3: Bumping manifests a=b2g-bump
B2G Bumper Bot <release+b2gbumper@mozilla.com> - Sat, 13 Sep 2014 09:17:45 -0700 - rev 205195
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Bumping manifests a=b2g-bump
bd7e2583438d800c011c912fc5d9192b98d473d5: Bumping manifests a=b2g-bump
B2G Bumper Bot <release+b2gbumper@mozilla.com> - Sat, 13 Sep 2014 06:57:36 -0700 - rev 205194
Push 8724 by philringnalda@gmail.com at Sun, 14 Sep 2014 18:46:29 +0000
Bumping manifests a=b2g-bump
(0) -100000 -30000 -10000 -3000 -1000 -300 -100 -50 -20 +20 +50 +100 +300 +1000 +3000 +10000 +30000 +100000 tip