compositor-worker: plumb element id and mutable properties

third_party/WebKit/Source/core/dom/Element.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Peter Kelly (pmk@post.com)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2007 David Smith (catfish.man@gmail.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012, 2013 Apple Inc. All rights reserved.
  *           (C) 2007 Eric Seidel (eric@webkit.org)
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 956 matching lines @@
 
     double scaledLeft = viewport->scrollPositionDouble().x();
     double scaledTop = viewport->scrollPositionDouble().y();
     if (scrollToOptions.hasLeft())
         scaledLeft = ScrollableArea::normalizeNonFiniteScroll(scrollToOptions.left()) * frame->pageZoomFactor();
     if (scrollToOptions.hasTop())
         scaledTop = ScrollableArea::normalizeNonFiniteScroll(scrollToOptions.top()) * frame->pageZoomFactor();
     viewport->setScrollPosition(DoublePoint(scaledLeft, scaledTop), ProgrammaticScroll, scrollBehavior);
 }
 
-void Element::incrementProxyCount()
+bool Element::hasCompositorProxy() const
 {
-    if (ensureElementRareData().incrementProxyCount() == 1)
+    return hasRareData() && elementRareData()->proxiedPropertyCounts();
+}
+
+void Element::incrementCompositorProxiedProperties(uint32_t mutableProperties)
+{
+    ElementRareData& rareData = ensureElementRareData();
+    if (!rareData.proxiedPropertyCounts())
+        setNeedsStyleRecalc(LocalStyleChange, StyleChangeReasonForTracing::create(StyleChangeReason::CompositorProxy));
+    rareData.incrementCompositorProxiedProperties(mutableProperties);
+}
+
+void Element::decrementCompositorProxiedProperties(uint32_t mutableProperties)
+{
+    ElementRareData& rareData = *elementRareData();
+    rareData.decrementCompositorProxiedProperties(mutableProperties);
+    if (!rareData.proxiedPropertyCounts())
         setNeedsStyleRecalc(LocalStyleChange, StyleChangeReasonForTracing::create(StyleChangeReason::CompositorProxy));
 }
 
-void Element::decrementProxyCount()
+uint32_t Element::compositorMutableProperties() const
 {
-    if (ensureElementRareData().decrementProxyCount() == 0)
-        setNeedsStyleRecalc(LocalStyleChange, StyleChangeReasonForTracing::create(StyleChangeReason::CompositorProxy));
+    if (!hasCompositorProxy())

[esprehn, 2015/12/07 20:54:24]

this technically is a few extra branches and function calls, I doubt it matters.
the most optimal would be:

if (!hasRareData())
  return WebCompositorMutablePropertyNone;
if (CompositorProxiedPropertySet* set =
elementRareData()->proxiedPropertyCounts())
  return set->proxiedProperties();
return WebCompositorMutablePropertyNone;

but you don't need to do that. :)

[Ian Vollick, 2015/12/08 00:16:44]

I did it!

+        return WebCompositorMutablePropertyNone;
+    return elementRareData()->proxiedPropertyCounts()->proxiedProperties();
 }
 
 bool Element::hasNonEmptyLayoutSize() const
 {
     document().updateLayoutIgnorePendingStylesheets();
 
     if (LayoutBoxModelObject* box = layoutBoxModelObject())
         return box->hasNonEmptyLayoutSize();
     return false;
 }
@@ skipping 683 matching lines @@
     if (ElementAnimations* elementAnimations = this->elementAnimations()) {
         elementAnimations->cssAnimations().maybeApplyPendingUpdate(this);
         elementAnimations->updateAnimationFlags(*style);
     }
 
     if (style->hasTransform()) {
         if (const StylePropertySet* inlineStyle = this->inlineStyle())
             style->setHasInlineTransform(inlineStyle->hasProperty(CSSPropertyTransform));
     }
 
-    if (hasRareData() && elementRareData()->proxyCount() > 0)
-        style->setHasCompositorProxy(true);
-
     document().didRecalculateStyleForElement();
     return style.release();
 }
 
 PassRefPtr<ComputedStyle> Element::originalStyleForLayoutObject()
 {
     ASSERT(document().inStyleRecalc());
     return document().ensureStyleResolver().styleForElement(this);
 }
 
@@ skipping 1913 matching lines @@
 {
 #if ENABLE(OILPAN)
     if (hasRareData())
         visitor->trace(elementRareData());
     visitor->trace(m_elementData);
 #endif
     ContainerNode::trace(visitor);
 }
 
 } // namespace blink