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

Unified Diff: third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositorTest.cpp

Issue 2637383006: Set layer element id when building layers in PaintArtifactCompositor. (Closed)
Patch Set: Sync to head. Created 3 years, 11 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: third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositorTest.cpp
diff --git a/third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositorTest.cpp b/third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositorTest.cpp
index c58bcde33d42102ce96836d6dd5809f1da515d7d..d0985d34611205069f8c8a1dee45ec55a3f52c7c 100644
--- a/third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositorTest.cpp
+++ b/third_party/WebKit/Source/platform/graphics/compositing/PaintArtifactCompositorTest.cpp
@@ -686,6 +686,7 @@ TEST_F(PaintArtifactCompositorTestWithPropertyTrees, OneScrollNode) {
EXPECT_EQ(expectedCompositorElementId, scrollNode.element_id);
EXPECT_EQ(scrollNode.id,
elementIdToScrollNodeIndex(expectedCompositorElementId));
+ EXPECT_EQ(expectedCompositorElementId, contentLayerAt(0)->element_id());
const cc::TransformTree& transformTree = propertyTrees().transform_tree;
const cc::TransformNode& transformNode =
@@ -751,6 +752,8 @@ TEST_F(PaintArtifactCompositorTestWithPropertyTrees, NestedScrollNodes) {
EXPECT_EQ(expectedCompositorElementIdB, scrollNodeB.element_id);
EXPECT_EQ(scrollNodeB.id,
elementIdToScrollNodeIndex(expectedCompositorElementIdB));
+ EXPECT_EQ(expectedCompositorElementIdA, contentLayerAt(0)->element_id());
+ EXPECT_EQ(expectedCompositorElementIdB, contentLayerAt(1)->element_id());
const cc::TransformTree& transformTree = propertyTrees().transform_tree;
const cc::TransformNode& transformNodeA =

Powered by Google App Engine
This is Rietveld 408576698