Add a WebDocument::watchCssSelectors(selectors)

Source/core/css/resolver/StyleBuilderCustom.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 2004-2005 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2006, 2007 Nicholas Shanks (webkit@nickshanks.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All rights reserved.
  * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org>
  * Copyright (C) 2007, 2008 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (c) 2011, Code Aurora Forum. All rights reserved.
@@ skipping 66 matching lines @@
 #include "core/css/ShadowValue.h"
 #include "core/css/StylePropertySet.h"
 #include "core/css/StylePropertyShorthand.h"
 #include "core/css/StyleSheetContents.h"
 #include "core/css/resolver/ElementStyleResources.h"
 #include "core/css/resolver/FilterOperationResolver.h"
 #include "core/css/resolver/StyleResolver.h"
 #include "core/css/resolver/StyleResolverState.h"
 #include "core/css/resolver/TransformBuilder.h"
 #include "core/css/resolver/ViewportStyleResolver.h"
+#include "core/dom/CSSSelectorWatch.h"
 #include "core/dom/DocumentStyleSheetCollection.h"
 #include "core/dom/Text.h"
 #include "core/dom/shadow/ShadowRoot.h"
 #include "core/page/Frame.h"
 #include "core/page/FrameView.h"
 #include "core/page/Page.h"
 #include "core/page/Settings.h"
 #include "core/platform/LinkHash.h"
 #include "core/platform/graphics/FontDescription.h"
 #include "core/platform/graphics/filters/custom/CustomFilterConstants.h"
@@ skipping 1603 matching lines @@
     }
     case CSSPropertyWebkitTapHighlightColor: {
         HANDLE_INHERIT_AND_INITIAL(tapHighlightColor, TapHighlightColor);
         if (!primitiveValue)
             break;
 
         Color col = state.document()->textLinkColors().colorFromPrimitiveValue(primitiveValue, state.style()->visitedDependentColor(CSSPropertyColor));
         state.style()->setTapHighlightColor(col);
         return;
     }
+    case CSSPropertyInternalCallback: {
+        if (isInherit || isInitial)
+            return;
+        if (primitiveValue) {
+            switch (primitiveValue->getValueID()) {
+            case CSSValueInternalPresence:
+                state.style()->addCallbackSelector(
+                    state.rule()->selectorList().selectorsText(),

[esprehn, 2013/07/17 07:14:32]

This dynamically recreates the text of the selector on every invocation, so for
the "div" case you're creating thousands of strings.

[Jeffrey Yasskin, 2013/08/02 01:34:07]

Right, but the "div" case didn't show up as significantly slower than the
"input[type=password]" case.

+                    CSSSelectorWatch::from(state.element()->document()));

[esprehn, 2013/07/17 07:14:32]

The hash table lookup hiding in here is going to hurt your perf numbers

[Jeffrey Yasskin, 2013/08/02 01:34:07]

A hash table lookup is generally cheaper than an allocation. I can try moving
this back to a field on Document to see if that improves anything (after trying
a couple other things), but I'd be surprised if it did.

+                return;
+            default:
+                break;
+            }
+        }
+        break;
+    }
     case CSSPropertyInvalid:
         return;
     // Directional properties are resolved by resolveDirectionAwareProperty() before the switch.
     case CSSPropertyWebkitBorderEndColor:
     case CSSPropertyWebkitBorderEndStyle:
     case CSSPropertyWebkitBorderEndWidth:
     case CSSPropertyWebkitBorderStartColor:
     case CSSPropertyWebkitBorderStartStyle:
     case CSSPropertyWebkitBorderStartWidth:
     case CSSPropertyWebkitBorderBeforeColor:
@@ skipping 834 matching lines @@
     case CSSPropertyMaskType: {
         HANDLE_SVG_INHERIT_AND_INITIAL(maskType, MaskType)
         if (primitiveValue)
             state.style()->accessSVGStyle()->setMaskType(*primitiveValue);
         break;
     }
     }
 }
 
 } // namespace WebCore