Avoid subtree style recalc for fullscreen pseudos.

Avoid subtree style recalc for fullscreen pseudos.

Do not let the presence of any of the following pseudo classes require a
sub-tree style recalc:

:-webkit-full-screen
:-webkit-full-screen-ancestor
:-webkit-full-screen-document

R=chrishtr@chromium.org,esprehn@chromium.org
BUG=425070
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=184441

[rune, 2014-10-24 08:46:10]

PTAL

[dglazkov, 2014-10-24 15:03:09]

dglazkov@chromium.org changed reviewers:
+ dglazkov@chromium.org

[dglazkov, 2014-10-24 15:03:09]

The CQ bit was checked by dglazkov@chromium.org

[dglazkov, 2014-10-24 15:03:10]

lgtm

[commit-bot: I haz the power, 2014-10-24 15:04:12]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/663023003/1

[commit-bot: I haz the power, 2014-10-24 15:04:32]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-10-24 15:04:33]

Failed to apply patch for Source/core/css/RuleFeature.cpp:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file Source/core/css/RuleFeature.cpp
  Hunk #1 succeeded at 118 (offset 1 line).
  Hunk #2 FAILED at 169.
  1 out of 2 hunks FAILED -- saving rejects to file
Source/core/css/RuleFeature.cpp.rej

Patch:       Source/core/css/RuleFeature.cpp
Index: Source/core/css/RuleFeature.cpp
diff --git a/Source/core/css/RuleFeature.cpp b/Source/core/css/RuleFeature.cpp
index
272b9b06ce26109250e0f41884055f8a8e5f32f2..b4808599875a4127e9e6a9c8395b1782913888fb
100644
--- a/Source/core/css/RuleFeature.cpp
+++ b/Source/core/css/RuleFeature.cpp
@@ -117,6 +117,9 @@ static bool supportsInvalidation(CSSSelector::PseudoType
type)
     case CSSSelector::PseudoScrollbarThumb:
     case CSSSelector::PseudoScrollbarTrack:
     case CSSSelector::PseudoScrollbarTrackPiece:
+    case CSSSelector::PseudoFullScreen:
+    case CSSSelector::PseudoFullScreenDocument:
+    case CSSSelector::PseudoFullScreenAncestor:
     case CSSSelector::PseudoInRange:
     case CSSSelector::PseudoOutOfRange:
     case CSSSelector::PseudoUnresolved:
@@ -166,9 +169,6 @@ static bool requiresSubtreeInvalidation(const CSSSelector&
selector)
     case CSSSelector::PseudoSingleButton:
     case CSSSelector::PseudoNoButton:
     case CSSSelector::PseudoSelection:
-    case CSSSelector::PseudoFullScreen:
-    case CSSSelector::PseudoFullScreenDocument:
-    case CSSSelector::PseudoFullScreenAncestor:
     case CSSSelector::PseudoUserAgentCustomElement:
     case CSSSelector::PseudoWebKitCustomElement:
     case CSSSelector::PseudoCue:

[rune, 2014-10-27 10:58:23]

The CQ bit was checked by rune@opera.com

[commit-bot: I haz the power, 2014-10-27 10:58:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/663023003/20001

[commit-bot: I haz the power, 2014-10-27 11:59:20]

Committed patchset #2 (id:20001) as 184441