compositor-worker: plumb element id and mutable properties

third_party/WebKit/Source/core/layout/compositing/PaintLayerCompositor.cpp

 /*
  * Copyright (C) 2009, 2010 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 45 matching lines @@
 #include "core/paint/TransformRecorder.h"
 #include "platform/RuntimeEnabledFeatures.h"
 #include "platform/ScriptForbiddenScope.h"
 #include "platform/TraceEvent.h"
 #include "platform/graphics/GraphicsLayer.h"
 #include "platform/graphics/paint/CullRect.h"
 #include "platform/graphics/paint/DrawingRecorder.h"
 #include "platform/graphics/paint/PaintController.h"
 #include "platform/graphics/paint/TransformDisplayItem.h"
 #include "public/platform/Platform.h"
+#include "public/platform/WebCompositorMutableProperties.h"
 
 namespace blink {
 
 PaintLayerCompositor::PaintLayerCompositor(LayoutView& layoutView)
     : m_layoutView(layoutView)
     , m_compositingReasonFinder(layoutView)
     , m_pendingUpdateType(CompositingUpdateNone)
     , m_hasAcceleratedCompositing(true)
     , m_compositing(false)
     , m_rootShouldAlwaysCompositeDirty(true)
@@ skipping 308 matching lines @@
                 for (ScrollableArea* scrollableArea : *scrollableAreas)
                     layersChanged |= scrollableArea->updateAfterCompositingChange();
             }
         }
 
         if (layersChanged)
             updateType = std::max(updateType, CompositingUpdateRebuildTree);
     }
 
     if (updateType != CompositingUpdateNone) {
+        if (RuntimeEnabledFeatures::compositorWorkerEnabled() && m_scrollLayer) {
+            if (Element* scrollingElement = m_layoutView.document().scrollingElement()) {

[esprehn, 2015/12/02 03:28:49]

why do we need special treatment for the document level scrolling element?
shouldn't this be generic to all scrolling elements?

[Ian Vollick, 2015/12/03 17:54:43]

Once we have skobes' root layer scrolling, this should be unnecessary. As it
stands, overflow scrolling and frame scrolling are treated differently today.

+                uint64_t elementId = 0;
+                uint32_t mutableProperties = WebCompositorMutablePropertyNone;
+                if (scrollingElement->hasCompositorProxy()) {
+                    elementId = DOMNodeIds::idForNode(scrollingElement);
+                    mutableProperties = (WebCompositorMutablePropertyScrollLeft | WebCompositorMutablePropertyScrollTop) & scrollingElement->compositorMutableProperties();
+                }
+                m_scrollLayer->setElementId(elementId);
+                m_scrollLayer->setCompositorMutableProperties(mutableProperties);
+            }
+        }
+
         GraphicsLayerUpdater updater;
         updater.update(*updateRoot, layersNeedingPaintInvalidation);
 
         if (updater.needsRebuildTree())
             updateType = std::max(updateType, CompositingUpdateRebuildTree);
 
 #if ENABLE(ASSERT)
         // FIXME: Move this check to the end of the compositing update.
         GraphicsLayerUpdater::assertNeedsToUpdateGraphicsLayerBitsCleared(*updateRoot);
 #endif
@@ skipping 788 matching lines @@
     } else if (graphicsLayer == m_scrollLayer.get()) {
         name = "Frame Scrolling Layer";
     } else {
         ASSERT_NOT_REACHED();
     }
 
     return name;
 }
 
 } // namespace blink