Add more specific metrics for main thread scrolling reasons

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 1011 matching lines @@
   scrollLayer = layoutObject->frameView()->layerForScrolling();
   ASSERT_TRUE(scrollLayer);
 
   webScrollLayer = scrollLayer->platformLayer();
   ASSERT_TRUE(webScrollLayer->scrollable());
   ASSERT_FALSE(
       webScrollLayer->mainThreadScrollingReasons() &
       MainThreadScrollingReason::kHasNonLayerViewportConstrainedObjects);
 }
 
+TEST_F(ScrollingCoordinatorTest, StyleRelatedMainThreadScrollingReason) {
+  registerMockedHttpURLLoad("two_transparent_scrollable_area.html");
+  navigateTo(m_baseURL + "two_transparent_scrollable_area.html");
+  webViewImpl()->settings()->setPreferCompositingToLCDTextEnabled(false);

[bokan, 2016/12/20 20:09:07]

Please add another test where setPreferCompositingToLCDTextEnabled is true and
make sure that we don't set a main thread scrolling reason.

[yigu, 2016/12/20 20:17:11]

Done.

+  forceFullCompositingUpdate();
+
+  FrameView* frameView = frame()->view();
+  ASSERT_TRUE(frameView);
+  ASSERT_TRUE(frameView->mainThreadScrollingReasons() &
+              MainThreadScrollingReason::kHasOpacity);
+
+  // Remove opacity from one of the scrollers.
+  // Still need to scroll on main thread.
+  Document* document = frame()->document();
+  Element* container = document->getElementById("scroller1");
+  DCHECK(container);
+
+  container->removeAttribute("class");
+  forceFullCompositingUpdate();
+
+  ASSERT_TRUE(frameView->mainThreadScrollingReasons() &
+              MainThreadScrollingReason::kHasOpacity);
+
+  // Remove opacity from the other layout would lead to

[bokan, 2016/12/20 20:09:07]

nit: layout->scroller

[yigu, 2016/12/20 20:17:11]

Done.

+  // scroll on impl.
+  container = document->getElementById("scroller2");
+  DCHECK(container);
+
+  container->removeAttribute("class");
+  forceFullCompositingUpdate();
+
+  ASSERT_FALSE(frameView->mainThreadScrollingReasons() &
+               MainThreadScrollingReason::kHasOpacity);
+
+  // Add opacity would again lead to scroll on main thread
+  container->setAttribute("class", "transparent", ASSERT_NO_EXCEPTION);
+  forceFullCompositingUpdate();
+
+  ASSERT_TRUE(frameView->mainThreadScrollingReasons() &
+              MainThreadScrollingReason::kHasOpacity);
+}
+
 }  // namespace blink