Fixed clip resize for document.rootScroller with inertTopControls

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

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/frame/BrowserControls.h"
 #include "core/frame/FrameHost.h"
 #include "core/frame/FrameView.h"
 #include "core/frame/RootFrameViewport.h"
 #include "core/html/HTMLFrameOwnerElement.h"
 #include "core/layout/LayoutBox.h"
 #include "core/layout/api/LayoutViewItem.h"
+#include "core/layout/compositing/CompositedLayerMapping.h"
 #include "core/layout/compositing/PaintLayerCompositor.h"
 #include "core/page/Page.h"
 #include "core/page/scrolling/RootScrollerController.h"
 #include "core/page/scrolling/TopDocumentRootScrollerController.h"
+#include "core/paint/PaintLayer.h"
 #include "core/paint/PaintLayerScrollableArea.h"
 #include "platform/testing/URLTestHelpers.h"
 #include "platform/testing/UnitTestHelpers.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebURLLoaderMockFactory.h"
 #include "public/web/WebCache.h"
 #include "public/web/WebConsoleMessage.h"
 #include "public/web/WebRemoteFrame.h"
 #include "public/web/WebScriptSource.h"
 #include "public/web/WebSettings.h"
@@ skipping 56 matching lines @@
     settings->setMainFrameResizesAreOrientationChanges(true);
   }
 
   void registerMockedHttpURLLoad(const std::string& fileName) {
     URLTestHelpers::registerMockedURLFromBaseURL(
         WebString::fromUTF8(m_baseURL.c_str()),
         WebString::fromUTF8(fileName.c_str()));
   }
 
   void executeScript(const WebString& code) {
-    mainWebFrame()->executeScript(WebScriptSource(code));
-    mainWebFrame()->view()->updateAllLifecyclePhases();
+    executeScript(code, *mainWebFrame());
+  }
+
+  void executeScript(const WebString& code, WebLocalFrame& frame) {
+    frame.executeScript(WebScriptSource(code));
+    frame.view()->updateAllLifecyclePhases();
     runPendingTasks();
   }
 
   WebViewImpl* webViewImpl() const { return m_helper.webView(); }
 
   FrameHost& frameHost() const {
     return m_helper.webView()->page()->frameHost();
   }
 
   LocalFrame* mainFrame() const {
@@ skipping 768 matching lines @@
 
     EXPECT_FALSE(
         childCompositor->rootContentLayer()->platformLayer()->masksToBounds());
     EXPECT_FALSE(
         childCompositor->rootGraphicsLayer()->platformLayer()->masksToBounds());
     EXPECT_FALSE(
         childCompositor->containerLayer()->platformLayer()->masksToBounds());
   }
 }
 
+// Tests that the clipping layer is resized on the root scroller element even
+// if the layout height doesn't change.
+TEST_F(RootScrollerTest, BrowserControlsResizeClippingLayer) {
+  bool oldInertTopControls = RuntimeEnabledFeatures::inertTopControlsEnabled();
+  RuntimeEnabledFeatures::setInertTopControlsEnabled(true);
+
+  initialize("root-scroller.html");
+  Element* container = mainFrame()->document()->getElementById("container");
+
+  {
+    NonThrowableExceptionState exceptionState;
+    mainFrame()->document()->setRootScroller(container, exceptionState);
+
+    mainFrameView()->updateAllLifecyclePhases();
+    ASSERT_EQ(toLayoutBox(container->layoutObject())->clientHeight(), 400);
+
+    GraphicsLayer* clipLayer = toLayoutBox(container->layoutObject())
+                                   ->layer()
+                                   ->compositedLayerMapping()
+                                   ->scrollingLayer();
+    ASSERT_EQ(clipLayer->size().height(), 400);
+    EXPECT_TRUE(toLayoutBox(container->layoutObject())
+                    ->layer()
+                    ->isGlobalRootScroller());
+  }
+
+  {
+    webViewImpl()->handleInputEvent(
+        generateTouchGestureEvent(WebInputEvent::GestureScrollBegin));
+
+    // Scrolling over the #container DIV should cause the browser controls to
+    // hide.
+    EXPECT_FLOAT_EQ(1, browserControls().shownRatio());
+    webViewImpl()->handleInputEvent(generateTouchGestureEvent(
+        WebInputEvent::GestureScrollUpdate, 0, -browserControls().height()));
+    EXPECT_FLOAT_EQ(0, browserControls().shownRatio());
+
+    webViewImpl()->handleInputEvent(
+        generateTouchGestureEvent(WebInputEvent::GestureScrollEnd));
+
+    webViewImpl()->resizeWithBrowserControls(IntSize(400, 450), 50, false);
+    mainFrameView()->updateAllLifecyclePhases();

[chrishtr, 2016/11/18 00:37:09]

Add a check before this line that the root scroller element does not have
needsLayout.

[bokan, 2016/11/18 22:30:35]

Done.

+
+    // Since inert top controls are enabled, the container should not have
+    // resized, however, the clip layer should.
+    EXPECT_EQ(toLayoutBox(container->layoutObject())->clientHeight(), 400);
+    GraphicsLayer* clipLayer = toLayoutBox(container->layoutObject())
+                                   ->layer()
+                                   ->compositedLayerMapping()
+                                   ->scrollingLayer();
+    EXPECT_EQ(clipLayer->size().height(), 450);
+  }
+
+  RuntimeEnabledFeatures::setInertTopControlsEnabled(oldInertTopControls);
+}
+
+// Tests that the clipping layer is resized on the root scroller element when
+// it's an iframe and even if the layout height doesn't change.
+TEST_F(RootScrollerTest, BrowserControlsResizeClippingLayerIFrame) {
+  bool oldInertTopControls = RuntimeEnabledFeatures::inertTopControlsEnabled();
+  RuntimeEnabledFeatures::setInertTopControlsEnabled(true);
+
+  initialize("root-scroller-iframe.html");
+
+  Element* iframe = mainFrame()->document()->getElementById("iframe");
+  LocalFrame* childFrame =
+      toLocalFrame(toHTMLFrameOwnerElement(iframe)->contentFrame());
+
+  PaintLayerCompositor* childPLC =
+      childFrame->view()->layoutViewItem().compositor();
+
+  // Give the iframe itself scrollable content and make it the root scroller.
+  {
+    NonThrowableExceptionState nonThrow;
+    mainFrame()->document()->setRootScroller(iframe, nonThrow);
+
+    WebLocalFrame* childWebFrame =
+        mainWebFrame()->firstChild()->toWebLocalFrame();
+    executeScript(
+        "document.getElementById('container').style.width = '300%';"
+        "document.getElementById('container').style.height = '300%';",
+        *childWebFrame);
+
+    mainFrameView()->updateAllLifecyclePhases();
+
+    // Some sanity checks to make sure the test is setup correctly.
+    ASSERT_EQ(childFrame->view()->visibleContentSize().height(), 400);
+    ASSERT_EQ(childPLC->containerLayer()->size().height(), 400);
+    ASSERT_EQ(childPLC->rootGraphicsLayer()->size().height(), 400);
+    ASSERT_TRUE(childFrame->view()->layer()->isGlobalRootScroller());
+  }
+
+  {
+    webViewImpl()->handleInputEvent(
+        generateTouchGestureEvent(WebInputEvent::GestureScrollBegin));
+
+    // Scrolling over the #container DIV should cause the browser controls to
+    // hide.
+    EXPECT_FLOAT_EQ(1, browserControls().shownRatio());
+    webViewImpl()->handleInputEvent(generateTouchGestureEvent(
+        WebInputEvent::GestureScrollUpdate, 0, -browserControls().height()));
+    EXPECT_FLOAT_EQ(0, browserControls().shownRatio());
+
+    webViewImpl()->handleInputEvent(
+        generateTouchGestureEvent(WebInputEvent::GestureScrollEnd));
+
+    webViewImpl()->resizeWithBrowserControls(IntSize(400, 450), 50, false);
+    mainFrameView()->updateAllLifecyclePhases();
+
+    // Since inert top controls are enabled, the iframe element should not have
+    // resized, however, its clip layer should resize to reveal content as the
+    // browser controls hide.
+    EXPECT_EQ(childFrame->view()->visibleContentSize().height(), 400);
+    EXPECT_EQ(childPLC->containerLayer()->size().height(), 450);
+    EXPECT_EQ(childPLC->rootGraphicsLayer()->size().height(), 450);
+  }
+
+  RuntimeEnabledFeatures::setInertTopControlsEnabled(oldInertTopControls);
+}
+
 // Tests that removing the root scroller element from the DOM resets the
 // effective root scroller without waiting for any lifecycle events.
 TEST_F(RootScrollerTest, RemoveRootScrollerFromDom) {
   initialize("root-scroller-iframe.html");
 
   {
     HTMLFrameOwnerElement* iframe = toHTMLFrameOwnerElement(
         mainFrame()->document()->getElementById("iframe"));
     Element* innerContainer =
         iframe->contentDocument()->getElementById("container");
@@ skipping 35 matching lines @@
   EXPECT_EQ(mainFrame()->document()->documentElement(),
             globalController.globalRootScroller());
   EXPECT_EQ(
       mainFrameView()->layoutViewportScrollableArea()->layerForScrolling(),
       globalController.rootScrollerLayer());
 }
 
 }  // namespace
 
 }  // namespace blink