Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1739)

Unified Diff: Source/core/rendering/compositing/GraphicsLayerTreeBuilder.cpp

Issue 303253004: Allow proper highlighting on universal overflow scroll. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: TestExpectations Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/rendering/compositing/GraphicsLayerTreeBuilder.cpp
diff --git a/Source/core/rendering/compositing/GraphicsLayerTreeBuilder.cpp b/Source/core/rendering/compositing/GraphicsLayerTreeBuilder.cpp
index 9826be1185ed18813e8972be622b3a63ceac7416..1188aed30df4f00a05bff16924dbbcd4fcffe59e 100644
--- a/Source/core/rendering/compositing/GraphicsLayerTreeBuilder.cpp
+++ b/Source/core/rendering/compositing/GraphicsLayerTreeBuilder.cpp
@@ -85,6 +85,9 @@ void GraphicsLayerTreeBuilder::rebuild(RenderLayer& layer, GraphicsLayerVector&
childList.append(currentCompositedLayerMapping->foregroundLayer());
}
+ if (hasCompositedLayerMapping && currentCompositedLayerMapping->scrollingBlockSelectionLayer())
Ian Vollick 2014/06/12 14:54:57 Please add a comment here explaining why this code
+ childList.append(currentCompositedLayerMapping->scrollingBlockSelectionLayer());
abarth-chromium 2014/06/13 04:12:02 Note: If you have this logic here, you need to sch
hartmanng 2014/06/13 14:51:38 Turns out we can get away without this part by fix
+
RenderLayerStackingNodeIterator iterator(*layer.stackingNode(), NormalFlowChildren | PositiveZOrderChildren);
while (RenderLayerStackingNode* curNode = iterator.next())
rebuild(*curNode->layer(), childList);

Powered by Google App Engine
This is Rietveld 408576698