Shifting layer position for sticky element to avoid passing unnessary variable to cc

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

Index: third_party/WebKit/Source/core/layout/compositing/CompositedLayerMappingTest.cpp
diff --git a/third_party/WebKit/Source/core/layout/compositing/CompositedLayerMappingTest.cpp b/third_party/WebKit/Source/core/layout/compositing/CompositedLayerMappingTest.cpp
index 2e1364f638a1237f4235c569fb50b73e143ebb22..d4a8954fb239b518be70fc7ba8621888111d6cd3 100644
--- a/third_party/WebKit/Source/core/layout/compositing/CompositedLayerMappingTest.cpp
+++ b/third_party/WebKit/Source/core/layout/compositing/CompositedLayerMappingTest.cpp
@@ -1495,11 +1495,6 @@ TEST_P(CompositedLayerMappingTest, StickyPositionMainThreadOffset) {
   CompositedLayerMapping* sticky_mapping =
       sticky_layer->GetCompositedLayerMapping();
   ASSERT_TRUE(sticky_mapping);
-  // Main thread offset for sticky should be "StickyTop - InnerPadding".
-  EXPECT_EQ(IntPoint(0, 15), IntPoint(sticky_mapping->MainGraphicsLayer()
-                                          ->ContentLayer()
-                                          ->Layer()
-                                          ->OffsetForStickyPosition()));
 
   // Now scroll the page - this should increase the main thread offset.
   LayoutBoxModelObject* scroller =
@@ -1513,11 +1508,6 @@ TEST_P(CompositedLayerMappingTest, StickyPositionMainThreadOffset) {
   EXPECT_TRUE(sticky_layer->NeedsCompositingInputsUpdate());
   GetDocument().View()->UpdateLifecycleToCompositingCleanPlusScrolling();
   EXPECT_FALSE(sticky_layer->NeedsCompositingInputsUpdate());
-
-  EXPECT_EQ(IntPoint(0, 115), IntPoint(sticky_mapping->MainGraphicsLayer()
-                                           ->ContentLayer()
-                                           ->Layer()
-                                           ->OffsetForStickyPosition()));
 }
 
 TEST_P(CompositedLayerMappingTest, StickyPositionNotSquashed) {
@@ -1561,4 +1551,71 @@ TEST_P(CompositedLayerMappingTest, StickyPositionNotSquashed) {
   EXPECT_EQ(kPaintsIntoOwnBacking, sticky3->GetCompositingState());
 }
 
+TEST_P(CompositedLayerMappingTest,
+       LayerPositionForStickyElementInCompositedScroller) {
+  SetBodyInnerHTML(
+      "<style>"
+      " .scroller { overflow: scroll; width: 200px; height: 600px; }"
+      " .composited { will-change:transform; }"
+      " .box { position: sticky; width: 185px; height: 50px; top: 0px; }"
+      " .container { width: 100%; height: 1000px; }"
+      "</style>"
+      "<div id='scroller' class='composited scroller'>"
+      " <div class='container'>"
+      "  <div id='sticky' class='box'></div>"
+      " </div>"
+      "</div>");
+
+  CompositedLayerMapping* mapping =
+      ToLayoutBlock(GetLayoutObjectByElementId("sticky"))
+          ->Layer()
+          ->GetCompositedLayerMapping();
+  ASSERT_TRUE(mapping);
+  GraphicsLayer* main_graphics_layer = mapping->MainGraphicsLayer();
+
+  PaintLayer* scroller =
+      ToLayoutBlock(GetLayoutObjectByElementId("scroller"))->Layer();
+  PaintLayerScrollableArea* scrollable_area = scroller->GetScrollableArea();
+  scrollable_area->ScrollToAbsolutePosition(
+      FloatPoint(scrollable_area->ScrollPosition().Y(), 100));
+  GetDocument().View()->UpdateAllLifecyclePhases();
+
+  EXPECT_FLOAT_EQ(0, main_graphics_layer->GetPosition().X());
+  EXPECT_FLOAT_EQ(0, main_graphics_layer->GetPosition().Y());
+}
+
+TEST_P(CompositedLayerMappingTest,
+       LayerPositionForStickyElementInNonCompositedScroller) {
+  SetBodyInnerHTML(
+      "<style>"
+      " .scroller { overflow: scroll; width: 200px; height: 600px; }"
+      " .composited { will-change:transform; }"
+      " .box { position: sticky; width: 185px; height: 50px; top: 0px; }"
+      " .container { width: 100%; height: 1000px; }"
+      "</style>"
+      "<div id='scroller' class='scroller'>"
+      " <div class='composited container'>"

[flackr, 2017/06/09 17:33:33]

Add this above too so the cases are identical other than the composited
scroller.

[yigu, 2017/06/09 17:41:16]

Done.

+      "  <div id='sticky' class='box'></div>"
+      " </div>"
+      "</div>");
+
+  CompositedLayerMapping* mapping =
+      ToLayoutBlock(GetLayoutObjectByElementId("sticky"))
+          ->Layer()
+          ->GetCompositedLayerMapping();
+  ASSERT_TRUE(mapping);
+  GraphicsLayer* main_graphics_layer = mapping->MainGraphicsLayer();
+
+  PaintLayer* scroller =
+      ToLayoutBlock(GetLayoutObjectByElementId("scroller"))->Layer();
+  PaintLayerScrollableArea* scrollable_area = scroller->GetScrollableArea();
+  ASSERT_TRUE(scrollable_area);
+  scrollable_area->ScrollToAbsolutePosition(
+      FloatPoint(scrollable_area->ScrollPosition().Y(), 100));
+  GetDocument().View()->UpdateAllLifecyclePhases();
+
+  EXPECT_FLOAT_EQ(0, main_graphics_layer->GetPosition().X());
+  EXPECT_FLOAT_EQ(100, main_graphics_layer->GetPosition().Y());
+}
+
 }  // namespace blink