Move logic of recording main thread scrolling reasons from cc to blink::ScrollManager

third_party/WebKit/Source/web/tests/ScrollingCoordinatorTest.cpp

 /*
  * Copyright (C) 2012 Google 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 1021 matching lines @@
   } else {
     ASSERT_TRUE(scrollLayer);
     webScrollLayer = scrollLayer->platformLayer();
     ASSERT_TRUE(webScrollLayer->scrollable());
     ASSERT_FALSE(
         webScrollLayer->mainThreadScrollingReasons() &
         MainThreadScrollingReason::kHasNonLayerViewportConstrainedObjects);
   }
 }
 
-class StyleRelatedMainThreadScrollingReasonTest
+class NonCompositedMainThreadScrollingReasonTest
     : public ScrollingCoordinatorTest {
   static const uint32_t m_LCDTextRelatedReasons =
       MainThreadScrollingReason::kHasOpacityAndLCDText |
       MainThreadScrollingReason::kHasTransformAndLCDText |
       MainThreadScrollingReason::kBackgroundNotOpaqueInRectAndLCDText;
 
  protected:
-  StyleRelatedMainThreadScrollingReasonTest() {
+  NonCompositedMainThreadScrollingReasonTest() {
     registerMockedHttpURLLoad("two_scrollable_area.html");
     navigateTo(m_baseURL + "two_scrollable_area.html");
   }
-  void testStyle(const std::string& target, const uint32_t reason) {
+  void testNonCompositedReasons(const std::string& target,
+                                const uint32_t reason) {
     webViewImpl()->settings()->setPreferCompositingToLCDTextEnabled(false);
     Document* document = frame()->document();
     Element* container = document->getElementById("scroller1");
-    container->setAttribute("class", target.c_str(), ASSERT_NO_EXCEPTION);
-    container = document->getElementById("scroller2");
+    DCHECK(container);
     container->setAttribute("class", target.c_str(), ASSERT_NO_EXCEPTION);
     forceFullCompositingUpdate();
 
+    PaintLayerScrollableArea* scrollableArea =
+        toLayoutBoxModelObject(container->layoutObject())->getScrollableArea();
+    ASSERT_TRUE(scrollableArea);
+    ASSERT_TRUE(scrollableArea->getNonCompositedMainThreadScrollingReasons() &
+                reason);
+
+    Element* container2 = document->getElementById("scroller2");
+    PaintLayerScrollableArea* scrollableArea2 =
+        toLayoutBoxModelObject(container2->layoutObject())->getScrollableArea();
+    ASSERT_TRUE(scrollableArea2);
+    // Different scrollable area should remain unaffected.
+    ASSERT_FALSE(scrollableArea2->getNonCompositedMainThreadScrollingReasons() &
+                 reason);
+
     FrameView* frameView = frame()->view();
     ASSERT_TRUE(frameView);
-    ASSERT_TRUE(frameView->mainThreadScrollingReasons() & reason);
+    ASSERT_FALSE(frameView->mainThreadScrollingReasons() & reason);
 
-    // Remove the target attribute from one of the scrollers.
-    // Still need to scroll on main thread.
-    container = document->getElementById("scroller1");
-    DCHECK(container);
-
+    // Remove attribute from the scroller 1 would lead to scroll on impl.
     container->removeAttribute("class");
     forceFullCompositingUpdate();
 
-    ASSERT_TRUE(frameView->mainThreadScrollingReasons() & reason);
-
-    // Remove attribute from the other scroller would lead to
-    // scroll on impl.
-    container = document->getElementById("scroller2");
-    DCHECK(container);
-
-    container->removeAttribute("class");
-    forceFullCompositingUpdate();
-
+    ASSERT_FALSE(scrollableArea->getNonCompositedMainThreadScrollingReasons() &
+                 reason);
     ASSERT_FALSE(frameView->mainThreadScrollingReasons() & reason);
 
     // Add target attribute would again lead to scroll on main thread
     container->setAttribute("class", target.c_str(), ASSERT_NO_EXCEPTION);
     forceFullCompositingUpdate();
 
-    ASSERT_TRUE(frameView->mainThreadScrollingReasons() & reason);
+    ASSERT_TRUE(scrollableArea->getNonCompositedMainThreadScrollingReasons() &
+                reason);
+    ASSERT_FALSE(frameView->mainThreadScrollingReasons() & reason);
 
     if ((reason & m_LCDTextRelatedReasons) &&
         !(reason & ~m_LCDTextRelatedReasons)) {

[bokan, 2017/04/07 21:05:57]

We always provide just one reason right? This if could probably just check the
first part of the &&

[yigu, 2017/04/10 20:39:05]

We may pass multiple reasons like the test LCDTextEnabledTest below.

       webViewImpl()->settings()->setPreferCompositingToLCDTextEnabled(true);
       forceFullCompositingUpdate();
+      ASSERT_FALSE(
+          scrollableArea->getNonCompositedMainThreadScrollingReasons() &
+          reason);
       ASSERT_FALSE(frameView->mainThreadScrollingReasons());
     }
   }
 };
 
 INSTANTIATE_TEST_CASE_P(All,
-                        StyleRelatedMainThreadScrollingReasonTest,
+                        NonCompositedMainThreadScrollingReasonTest,
                         ::testing::Bool());
 
-// TODO(yigu): This test and all other style realted main thread scrolling
-// reason tests below have been disabled due to https://crbug.com/701355.
-TEST_P(StyleRelatedMainThreadScrollingReasonTest, DISABLED_TransparentTest) {
-  testStyle("transparent", MainThreadScrollingReason::kHasOpacityAndLCDText);
+TEST_P(NonCompositedMainThreadScrollingReasonTest, TransparentTest) {
+  testNonCompositedReasons("transparent",
+                           MainThreadScrollingReason::kHasOpacityAndLCDText);
 }
 
-TEST_P(StyleRelatedMainThreadScrollingReasonTest, DISABLED_TransformTest) {
-  testStyle("transform", MainThreadScrollingReason::kHasTransformAndLCDText);
+TEST_P(NonCompositedMainThreadScrollingReasonTest, TransformTest) {
+  testNonCompositedReasons("transform",
+                           MainThreadScrollingReason::kHasTransformAndLCDText);
 }
 
-TEST_P(StyleRelatedMainThreadScrollingReasonTest,
-       DISABLED_BackgroundNotOpaqueTest) {
-  testStyle("background-not-opaque",
-            MainThreadScrollingReason::kBackgroundNotOpaqueInRectAndLCDText);
+TEST_P(NonCompositedMainThreadScrollingReasonTest, BackgroundNotOpaqueTest) {
+  testNonCompositedReasons(
+      "background-not-opaque",
+      MainThreadScrollingReason::kBackgroundNotOpaqueInRectAndLCDText);
 }
 
-TEST_P(StyleRelatedMainThreadScrollingReasonTest, DISABLED_BorderRadiusTest) {
-  testStyle("border-radius", MainThreadScrollingReason::kHasBorderRadius);
+TEST_P(NonCompositedMainThreadScrollingReasonTest, BorderRadiusTest) {
+  testNonCompositedReasons("border-radius",
+                           MainThreadScrollingReason::kHasBorderRadius);
 }
 
-TEST_P(StyleRelatedMainThreadScrollingReasonTest, DISABLED_ClipTest) {
-  testStyle("clip", MainThreadScrollingReason::kHasClipRelatedProperty);
+TEST_P(NonCompositedMainThreadScrollingReasonTest, ClipTest) {
+  testNonCompositedReasons("clip",
+                           MainThreadScrollingReason::kHasClipRelatedProperty);
 }
 
-TEST_P(StyleRelatedMainThreadScrollingReasonTest, DISABLED_ClipPathTest) {
+TEST_P(NonCompositedMainThreadScrollingReasonTest, ClipPathTest) {
   uint32_t reason = MainThreadScrollingReason::kHasClipRelatedProperty;

[bokan, 2017/04/07 21:05:57]

reason -> clipReason or similar.

[yigu, 2017/04/10 20:39:05]

Done.

   webViewImpl()->settings()->setPreferCompositingToLCDTextEnabled(false);
   Document* document = frame()->document();
   // Test ancestor with ClipPath
   Element* element = document->body();
   DCHECK(element);
   element->setAttribute(HTMLNames::styleAttr,
                         "clip-path:circle(115px at 20px 20px);");
+  Element* container = document->getElementById("scroller1");
+  DCHECK(container);
   forceFullCompositingUpdate();
 
+  PaintLayerScrollableArea* scrollableArea =
+      toLayoutBoxModelObject(container->layoutObject())->getScrollableArea();
+  ASSERT_TRUE(scrollableArea);
+  ASSERT_TRUE(scrollableArea->getNonCompositedMainThreadScrollingReasons() &
+              reason);
+
   FrameView* frameView = frame()->view();
   ASSERT_TRUE(frameView);
-  ASSERT_TRUE(frameView->mainThreadScrollingReasons() & reason);
+  ASSERT_FALSE(frameView->mainThreadScrollingReasons() & reason);
 
   // Remove clip path from ancestor.
   element->removeAttribute(HTMLNames::styleAttr);
   forceFullCompositingUpdate();
 
+  ASSERT_FALSE(scrollableArea->getNonCompositedMainThreadScrollingReasons() &
+               reason);
   ASSERT_FALSE(frameView->mainThreadScrollingReasons() & reason);
 
   // Test descendant with ClipPath
   element = document->getElementById("content1");
   DCHECK(element);
   element->setAttribute(HTMLNames::styleAttr,
                         "clip-path:circle(115px at 20px 20px);");
   forceFullCompositingUpdate();
-  ASSERT_TRUE(frameView->mainThreadScrollingReasons() & reason);
+  ASSERT_TRUE(scrollableArea->getNonCompositedMainThreadScrollingReasons() &
+              reason);
+  ASSERT_FALSE(frameView->mainThreadScrollingReasons() & reason);
 
   // Remove clip path from descendant.
   element->removeAttribute(HTMLNames::styleAttr);
   forceFullCompositingUpdate();
+  ASSERT_FALSE(scrollableArea->getNonCompositedMainThreadScrollingReasons() &
+               reason);
   ASSERT_FALSE(frameView->mainThreadScrollingReasons() & reason);
 }
 
-TEST_P(StyleRelatedMainThreadScrollingReasonTest, DISABLED_LCDTextEnabledTest) {
-  testStyle("transparent border-radius",
-            MainThreadScrollingReason::kHasOpacityAndLCDText |
-                MainThreadScrollingReason::kHasBorderRadius);
+TEST_P(NonCompositedMainThreadScrollingReasonTest, LCDTextEnabledTest) {
+  testNonCompositedReasons("transparent border-radius",
+                           MainThreadScrollingReason::kHasOpacityAndLCDText |
+                               MainThreadScrollingReason::kHasBorderRadius);
 }
 
-TEST_P(StyleRelatedMainThreadScrollingReasonTest, DISABLED_BoxShadowTest) {
-  testStyle("box-shadow",
-            MainThreadScrollingReason::kHasBoxShadowFromNonRootLayer);
+TEST_P(NonCompositedMainThreadScrollingReasonTest, BoxShadowTest) {
+  testNonCompositedReasons(
+      "box-shadow", MainThreadScrollingReason::kHasBoxShadowFromNonRootLayer);
+}
+
+TEST_P(NonCompositedMainThreadScrollingReasonTest,
+       CompositedWithLCDTextRelatedReasonsTest) {
+  // With "will-change:transform" we composite elements with
+  // LCDTextRelatedReasons only. For elements with other
+  // NonCompositedReasons, we don't create scrollingLayer for their
+  // CompositedLayerMapping therefore they don't get composited.
+  webViewImpl()->settings()->setPreferCompositingToLCDTextEnabled(false);
+  Document* document = frame()->document();
+  Element* container = document->getElementById("scroller1");
+  DCHECK(container);
+  container->setAttribute("class", "composited transparent",
+                          ASSERT_NO_EXCEPTION);
+  forceFullCompositingUpdate();
+
+  PaintLayerScrollableArea* scrollableArea =
+      toLayoutBoxModelObject(container->layoutObject())->getScrollableArea();
+  ASSERT_TRUE(scrollableArea);
+  ASSERT_FALSE(scrollableArea->getNonCompositedMainThreadScrollingReasons() &

[bokan, 2017/04/07 21:05:57]

Just ASSERT_FALSE(getNonCompositedMainThreadScrollingReasons())

[yigu, 2017/04/10 20:39:05]

Done.

+               MainThreadScrollingReason::kHasOpacityAndLCDText);
+
+  Element* container2 = document->getElementById("scroller2");
+  DCHECK(container);
+  container2->setAttribute("class", "composited border-radius",
+                           ASSERT_NO_EXCEPTION);
+  forceFullCompositingUpdate();
+  PaintLayerScrollableArea* scrollableArea2 =
+      toLayoutBoxModelObject(container2->layoutObject())->getScrollableArea();
+  ASSERT_TRUE(scrollableArea2);
+  ASSERT_TRUE(scrollableArea2->getNonCompositedMainThreadScrollingReasons() &
+              MainThreadScrollingReason::kHasBorderRadius);
 }
 
 }  // namespace blink