reflow comments in web/tests

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

 // Copyright 2014 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/VisualViewport.h"
 
 #include "core/dom/Document.h"
 #include "core/frame/FrameHost.h"
 #include "core/frame/FrameView.h"
 #include "core/frame/LocalFrame.h"
@@ skipping 411 matching lines @@
 
   navigateTo("about:blank");
   webViewImpl()->updateAllLifecyclePhases();
 
   VisualViewport& visualViewport =
       frame()->page()->frameHost().visualViewport();
   EXPECT_FLOAT_SIZE_EQ(FloatSize(320, 240),
                        visualViewport.containerLayer()->size());
 }
 
-// Make sure that the visibleRect method acurately reflects the scale and scroll location
-// of the viewport.
+// Make sure that the visibleRect method acurately reflects the scale and scroll
+// location of the viewport.
 TEST_P(ParameterizedVisualViewportTest, TestVisibleRect) {
   initializeWithDesktopSettings();
   webViewImpl()->resize(IntSize(320, 240));
 
   navigateTo("about:blank");
   forceFullCompositingUpdate();
 
   VisualViewport& visualViewport =
       frame()->page()->frameHost().visualViewport();
 
@@ skipping 47 matching lines @@
 
   VisualViewport& visualViewport =
       frame()->page()->frameHost().visualViewport();
 
   // Scale the viewport to 2X and move it.
   visualViewport.setScale(2);
   visualViewport.setLocation(FloatPoint(10, 15));
   EXPECT_FLOAT_RECT_EQ(FloatRect(10, 15, 50, 200),
                        visualViewport.visibleRectInDocument());
 
-  // Scroll the layout viewport. Ensure its offset is reflected in visibleRectInDocument().
+  // Scroll the layout viewport. Ensure its offset is reflected in
+  // visibleRectInDocument().
   FrameView& frameView = *webViewImpl()->mainFrameImpl()->frameView();
   frameView.layoutViewportScrollableArea()->setScrollPosition(
       DoublePoint(40, 100), ProgrammaticScroll);
   EXPECT_FLOAT_RECT_EQ(FloatRect(50, 115, 50, 200),
                        visualViewport.visibleRectInDocument());
 }
 
 TEST_P(ParameterizedVisualViewportTest,
        TestFractionalScrollOffsetIsNotOverwritten) {
   bool origFractionalOffsetsEnabled =
@@ skipping 13 matching lines @@
       DoublePoint(10, 30.5), CompositorScroll);
 
   EXPECT_EQ(
       30.5,
       frameView.layoutViewportScrollableArea()->scrollPositionDouble().y());
 
   RuntimeEnabledFeatures::setFractionalScrollOffsetsEnabled(
       origFractionalOffsetsEnabled);
 }
 
-// Test that the viewport's scroll offset is always appropriately bounded such that the
-// visual viewport always stays within the bounds of the main frame.
+// Test that the viewport's scroll offset is always appropriately bounded such
+// that the visual viewport always stays within the bounds of the main frame.
 TEST_P(ParameterizedVisualViewportTest, TestOffsetClamping) {
   initializeWithDesktopSettings();
   webViewImpl()->resize(IntSize(320, 240));
 
   navigateTo("about:blank");
   forceFullCompositingUpdate();
 
-  // Visual viewport should be initialized to same size as frame so no scrolling possible.
+  // Visual viewport should be initialized to same size as frame so no scrolling
+  // possible.
   VisualViewport& visualViewport =
       frame()->page()->frameHost().visualViewport();
   EXPECT_FLOAT_POINT_EQ(FloatPoint(0, 0),
                         visualViewport.visibleRect().location());
 
   visualViewport.setLocation(FloatPoint(-1, -2));
   EXPECT_FLOAT_POINT_EQ(FloatPoint(0, 0),
                         visualViewport.visibleRect().location());
 
   visualViewport.setLocation(FloatPoint(100, 200));
@@ skipping 11 matching lines @@
   EXPECT_FLOAT_POINT_EQ(location, visualViewport.visibleRect().location());
 
   visualViewport.setLocation(FloatPoint(1000, 2000));
   EXPECT_FLOAT_POINT_EQ(FloatPoint(160, 120),
                         visualViewport.visibleRect().location());
 
   visualViewport.setLocation(FloatPoint(-1000, -2000));
   EXPECT_FLOAT_POINT_EQ(FloatPoint(0, 0),
                         visualViewport.visibleRect().location());
 
-  // Make sure offset gets clamped on scale out. Scale to 1.25 so the viewport is 256x192.
+  // Make sure offset gets clamped on scale out. Scale to 1.25 so the viewport
+  // is 256x192.
   visualViewport.setLocation(FloatPoint(160, 120));
   visualViewport.setScale(1.25);
   EXPECT_FLOAT_POINT_EQ(FloatPoint(64, 48),
                         visualViewport.visibleRect().location());
 
   // Scale out smaller than 1.
   visualViewport.setScale(0.25);
   EXPECT_FLOAT_POINT_EQ(FloatPoint(0, 0),
                         visualViewport.visibleRect().location());
 }
 
-// Test that the viewport can be scrolled around only within the main frame in the presence
-// of viewport resizes, as would be the case if the on screen keyboard came up.
+// Test that the viewport can be scrolled around only within the main frame in
+// the presence of viewport resizes, as would be the case if the on screen
+// keyboard came up.
 TEST_P(ParameterizedVisualViewportTest, TestOffsetClampingWithResize) {
   initializeWithDesktopSettings();
   webViewImpl()->resize(IntSize(320, 240));
 
   navigateTo("about:blank");
   forceFullCompositingUpdate();
 
-  // Visual viewport should be initialized to same size as frame so no scrolling possible.
+  // Visual viewport should be initialized to same size as frame so no scrolling
+  // possible.
   VisualViewport& visualViewport =
       frame()->page()->frameHost().visualViewport();
   EXPECT_FLOAT_POINT_EQ(FloatPoint(0, 0),
                         visualViewport.visibleRect().location());
 
-  // Shrink the viewport vertically. The resize shouldn't affect the location, but it
-  // should allow vertical scrolling.
+  // Shrink the viewport vertically. The resize shouldn't affect the location,
+  // but it should allow vertical scrolling.
   visualViewport.setSize(IntSize(320, 200));
   EXPECT_FLOAT_POINT_EQ(FloatPoint(0, 0),
                         visualViewport.visibleRect().location());
   visualViewport.setLocation(FloatPoint(10, 20));
   EXPECT_FLOAT_POINT_EQ(FloatPoint(0, 20),
                         visualViewport.visibleRect().location());
   visualViewport.setLocation(FloatPoint(0, 100));
   EXPECT_FLOAT_POINT_EQ(FloatPoint(0, 40),
                         visualViewport.visibleRect().location());
   visualViewport.setLocation(FloatPoint(0, 10));
@@ skipping 31 matching lines @@
   EXPECT_FLOAT_POINT_EQ(FloatPoint(40, 40),
                         visualViewport.visibleRect().location());
   visualViewport.setLocation(FloatPoint(10, 3));
   EXPECT_FLOAT_POINT_EQ(FloatPoint(10, 3),
                         visualViewport.visibleRect().location());
   visualViewport.setLocation(FloatPoint(-10, -4));
   EXPECT_FLOAT_POINT_EQ(FloatPoint(0, 0),
                         visualViewport.visibleRect().location());
 }
 
-// Test that the viewport is scrollable but bounded appropriately within the main frame
-// when we apply both scaling and resizes.
+// Test that the viewport is scrollable but bounded appropriately within the
+// main frame when we apply both scaling and resizes.
 TEST_P(ParameterizedVisualViewportTest, TestOffsetClampingWithResizeAndScale) {
   initializeWithDesktopSettings();
   webViewImpl()->resize(IntSize(320, 240));
 
   navigateTo("about:blank");
   forceFullCompositingUpdate();
 
-  // Visual viewport should be initialized to same size as WebView so no scrolling possible.
+  // Visual viewport should be initialized to same size as WebView so no
+  // scrolling possible.
   VisualViewport& visualViewport =
       frame()->page()->frameHost().visualViewport();
   EXPECT_FLOAT_POINT_EQ(FloatPoint(0, 0),
                         visualViewport.visibleRect().location());
 
   // Zoom in to 2X so we can scroll the viewport to 160x120.
   visualViewport.setScale(2);
   visualViewport.setLocation(FloatPoint(200, 200));
   EXPECT_FLOAT_POINT_EQ(FloatPoint(160, 120),
                         visualViewport.visibleRect().location());
 
-  // Now resize the viewport to make it 10px smaller. Since we're zoomed in by 2X it should
-  // allow us to scroll by 5px more.
+  // Now resize the viewport to make it 10px smaller. Since we're zoomed in by
+  // 2X it should allow us to scroll by 5px more.
   visualViewport.setSize(IntSize(310, 230));
   visualViewport.setLocation(FloatPoint(200, 200));
   EXPECT_FLOAT_POINT_EQ(FloatPoint(165, 125),
                         visualViewport.visibleRect().location());
 
-  // The viewport can be larger than the main frame (currently 320, 240) though typically
-  // the scale will be clamped to prevent it from actually being larger.
+  // The viewport can be larger than the main frame (currently 320, 240) though
+  // typically the scale will be clamped to prevent it from actually being
+  // larger.
   visualViewport.setSize(IntSize(330, 250));
   EXPECT_SIZE_EQ(IntSize(330, 250), visualViewport.size());
 
   // Resize both the viewport and the frame to be larger.
   webViewImpl()->resize(IntSize(640, 480));
   webViewImpl()->updateAllLifecyclePhases();
   EXPECT_SIZE_EQ(IntSize(webViewImpl()->size()), visualViewport.size());
   EXPECT_SIZE_EQ(IntSize(webViewImpl()->size()),
                  frame()->view()->frameRect().size());
   visualViewport.setLocation(FloatPoint(1000, 1000));
   EXPECT_FLOAT_POINT_EQ(FloatPoint(320, 240),
                         visualViewport.visibleRect().location());
 
-  // Make sure resizing the viewport doesn't change its offset if the resize doesn't make
-  // the viewport go out of bounds.
+  // Make sure resizing the viewport doesn't change its offset if the resize
+  // doesn't make the viewport go out of bounds.
   visualViewport.setLocation(FloatPoint(200, 200));
   visualViewport.setSize(IntSize(880, 560));
   EXPECT_FLOAT_POINT_EQ(FloatPoint(200, 200),
                         visualViewport.visibleRect().location());
 }
 
-// The main FrameView's size should be set such that its the size of the visual viewport
-// at minimum scale. If there's no explicit minimum scale set, the FrameView should be
-// set to the content width and height derived by the aspect ratio.
+// The main FrameView's size should be set such that its the size of the visual
+// viewport at minimum scale. If there's no explicit minimum scale set, the
+// FrameView should be set to the content width and height derived by the aspect
+// ratio.
 TEST_P(ParameterizedVisualViewportTest, TestFrameViewSizedToContent) {
   initializeWithAndroidSettings();
   webViewImpl()->resize(IntSize(320, 240));
 
   registerMockedHttpURLLoad("200-by-300-viewport.html");
   navigateTo(m_baseURL + "200-by-300-viewport.html");
 
   webViewImpl()->resize(IntSize(600, 800));
   webViewImpl()->updateAllLifecyclePhases();
 
-  // Note: the size is ceiled and should match the behavior in CC's LayerImpl::bounds().
+  // Note: the size is ceiled and should match the behavior in CC's
+  // LayerImpl::bounds().
   EXPECT_SIZE_EQ(
       IntSize(200, 267),
       webViewImpl()->mainFrameImpl()->frameView()->frameRect().size());
 }
 
-// The main FrameView's size should be set such that its the size of the visual viewport
-// at minimum scale. On Desktop, the minimum scale is set at 1 so make sure the FrameView
-// is sized to the viewport.
+// The main FrameView's size should be set such that its the size of the visual
+// viewport at minimum scale. On Desktop, the minimum scale is set at 1 so make
+// sure the FrameView is sized to the viewport.
 TEST_P(ParameterizedVisualViewportTest, TestFrameViewSizedToMinimumScale) {
   initializeWithDesktopSettings();
   webViewImpl()->resize(IntSize(320, 240));
 
   registerMockedHttpURLLoad("200-by-300.html");
   navigateTo(m_baseURL + "200-by-300.html");
 
   webViewImpl()->resize(IntSize(100, 160));
   webViewImpl()->updateAllLifecyclePhases();
 
   EXPECT_SIZE_EQ(
       IntSize(100, 160),
       webViewImpl()->mainFrameImpl()->frameView()->frameRect().size());
 }
 
-// Test that attaching a new frame view resets the size of the inner viewport scroll
-// layer. crbug.com/423189.
+// Test that attaching a new frame view resets the size of the inner viewport
+// scroll layer. crbug.com/423189.
 TEST_P(ParameterizedVisualViewportTest,
        TestAttachingNewFrameSetsInnerScrollLayerSize) {
   initializeWithAndroidSettings();
   webViewImpl()->resize(IntSize(320, 240));
 
   // Load a wider page first, the navigation should resize the scroll layer to
   // the smaller size on the second navigation.
   registerMockedHttpURLLoad("content-width-1000.html");
   navigateTo(m_baseURL + "content-width-1000.html");
   webViewImpl()->updateAllLifecyclePhases();
@@ skipping 12 matching lines @@
   navigateTo(m_baseURL + "viewport-device-width.html");
 
   // Ensure the scroll layer matches the frame view's size.
   EXPECT_SIZE_EQ(FloatSize(320, 240), visualViewport.scrollLayer()->size());
 
   // Ensure the location and scale were reset.
   EXPECT_POINT_EQ(FloatPoint(), visualViewport.location());
   EXPECT_EQ(1, visualViewport.scale());
 }
 
-// The main FrameView's size should be set such that its the size of the visual viewport
-// at minimum scale. Test that the FrameView is appropriately sized in the presence
-// of a viewport <meta> tag.
+// The main FrameView's size should be set such that its the size of the visual
+// viewport at minimum scale. Test that the FrameView is appropriately sized in
+// the presence of a viewport <meta> tag.
 TEST_P(ParameterizedVisualViewportTest,
        TestFrameViewSizedToViewportMetaMinimumScale) {
   initializeWithAndroidSettings();
   webViewImpl()->resize(IntSize(320, 240));
 
   registerMockedHttpURLLoad("200-by-300-min-scale-2.html");
   navigateTo(m_baseURL + "200-by-300-min-scale-2.html");
 
   webViewImpl()->resize(IntSize(100, 160));
   webViewImpl()->updateAllLifecyclePhases();
@@ skipping 14 matching lines @@
 
   webViewImpl()->enableAutoResizeMode(WebSize(10, 10), WebSize(1000, 1000));
 
   registerMockedHttpURLLoad("200-by-300.html");
   navigateTo(m_baseURL + "200-by-300.html");
 
   EXPECT_SIZE_EQ(IntSize(200, 300),
                  frame()->page()->frameHost().visualViewport().size());
 }
 
-// Test that the text selection handle's position accounts for the visual viewport.
+// Test that the text selection handle's position accounts for the visual
+// viewport.
 TEST_P(ParameterizedVisualViewportTest, TestTextSelectionHandles) {
   initializeWithDesktopSettings();
   webViewImpl()->resize(IntSize(500, 800));
 
   registerMockedHttpURLLoad("pinch-viewport-input-field.html");
   navigateTo(m_baseURL + "pinch-viewport-input-field.html");
 
   VisualViewport& visualViewport =
       frame()->page()->frameHost().visualViewport();
   webViewImpl()->setInitialFocus(false);
 
   WebRect originalAnchor;
   WebRect originalFocus;
   webViewImpl()->selectionBounds(originalAnchor, originalFocus);
 
   webViewImpl()->setPageScaleFactor(2);
   visualViewport.setLocation(FloatPoint(100, 400));
 
   WebRect anchor;
   WebRect focus;
   webViewImpl()->selectionBounds(anchor, focus);
 
   IntPoint expected(IntRect(originalAnchor).location());
   expected.moveBy(-flooredIntPoint(visualViewport.visibleRect().location()));
   expected.scale(visualViewport.scale(), visualViewport.scale());
 
   EXPECT_POINT_EQ(expected, IntRect(anchor).location());
   EXPECT_POINT_EQ(expected, IntRect(focus).location());
 
-  // FIXME(bokan) - http://crbug.com/364154 - Figure out how to test text selection
-  // as well rather than just carret.
+  // FIXME(bokan) - http://crbug.com/364154 - Figure out how to test text
+  // selection as well rather than just carret.
 }
 
-// Test that the HistoryItem for the page stores the visual viewport's offset and scale.
+// Test that the HistoryItem for the page stores the visual viewport's offset
+// and scale.
 TEST_P(ParameterizedVisualViewportTest, TestSavedToHistoryItem) {
   initializeWithDesktopSettings();
   webViewImpl()->resize(IntSize(200, 300));
   webViewImpl()->updateAllLifecyclePhases();
 
   registerMockedHttpURLLoad("200-by-300.html");
   navigateTo(m_baseURL + "200-by-300.html");
 
   EXPECT_FLOAT_POINT_EQ(FloatPoint(0, 0),
                         toLocalFrame(webViewImpl()->page()->mainFrame())
@@ skipping 38 matching lines @@
                                     WebCachePolicy::UseProtocolCachePolicy);
 
   VisualViewport& visualViewport =
       frame()->page()->frameHost().visualViewport();
   EXPECT_EQ(2, visualViewport.scale());
 
   EXPECT_FLOAT_POINT_EQ(FloatPoint(100, 120),
                         visualViewport.visibleRect().location());
 }
 
-// Test restoring a HistoryItem without the visual viewport offset falls back to distributing
-// the scroll offset between the main frame and the visual viewport.
+// Test restoring a HistoryItem without the visual viewport offset falls back to
+// distributing the scroll offset between the main frame and the visual
+// viewport.
 TEST_F(VisualViewportTest, TestRestoredFromLegacyHistoryItem) {
   initializeWithDesktopSettings();
   webViewImpl()->resize(IntSize(100, 150));
 
   registerMockedHttpURLLoad("200-by-300-viewport.html");
 
   WebHistoryItem item;
   item.initialize();
   WebURL destinationURL(
       URLTestHelpers::toKURL(m_baseURL + "200-by-300-viewport.html"));
   item.setURLString(destinationURL.string());
-  // (-1, -1) will be used if the HistoryItem is an older version prior to having
-  // visual viewport scroll offset.
+  // (-1, -1) will be used if the HistoryItem is an older version prior to
+  // having visual viewport scroll offset.
   item.setVisualViewportScrollOffset(WebFloatPoint(-1, -1));
   item.setScrollOffset(WebPoint(120, 180));
   item.setPageScaleFactor(2);
 
   FrameTestHelpers::loadHistoryItem(webViewImpl()->mainFrame(), item,
                                     WebHistoryDifferentDocumentLoad,
                                     WebCachePolicy::UseProtocolCachePolicy);
 
   VisualViewport& visualViewport =
       frame()->page()->frameHost().visualViewport();
@@ skipping 61 matching lines @@
   // the range and its end). Do a sanity check that the expected text is
   // selected
   mainFrame->executeScript(WebScriptSource("selectRange();"));
   EXPECT_EQ("ir", mainFrame->toWebLocalFrame()->selectionAsText().utf8());
 
   webViewImpl()->selectionBounds(baseRect, extentRect);
   WebPoint initialPoint(baseRect.x, baseRect.y);
   WebPoint endPoint(extentRect.x, extentRect.y);
 
   // Move the visual viewport over and make the selection in the same
-  // screen-space location. The selection should change to two characters to
-  // the right and down one line.
+  // screen-space location. The selection should change to two characters to the
+  // right and down one line.
   VisualViewport& visualViewport =
       frame()->page()->frameHost().visualViewport();
   visualViewport.move(FloatPoint(60, 25));
   mainFrame->toWebLocalFrame()->moveRangeSelection(initialPoint, endPoint);
   EXPECT_EQ("t ", mainFrame->toWebLocalFrame()->selectionAsText().utf8());
 }
 
-// Test that the scrollFocusedEditableElementIntoRect method works with the visual viewport.
+// Test that the scrollFocusedEditableElementIntoRect method works with the
+// visual viewport.
 TEST_P(ParameterizedVisualViewportTest,
        DISABLED_TestScrollFocusedEditableElementIntoRect) {
   initializeWithDesktopSettings();
   webViewImpl()->resize(IntSize(500, 300));
 
   registerMockedHttpURLLoad("pinch-viewport-input-field.html");
   navigateTo(m_baseURL + "pinch-viewport-input-field.html");
 
   VisualViewport& visualViewport =
       frame()->page()->frameHost().visualViewport();
@@ skipping 12 matching lines @@
   webViewImpl()->resizeVisualViewport(IntSize(500, 300));
   visualViewport.setLocation(FloatPoint(0, 0));
 
   webViewImpl()->setPageScaleFactor(2);
   webViewImpl()->scrollFocusedEditableElementIntoRect(IntRect(0, 0, 500, 200));
   EXPECT_POINT_EQ(IntPoint(0, frame()->view()->maximumScrollPosition().y()),
                   frame()->view()->scrollPosition());
   EXPECT_FLOAT_POINT_EQ(FloatPoint(125, 150),
                         visualViewport.visibleRect().location());
 
-  // Once more but make sure that we don't move the visual viewport unless necessary.
+  // Once more but make sure that we don't move the visual viewport unless
+  // necessary.
   registerMockedHttpURLLoad("pinch-viewport-input-field-long-and-wide.html");
   navigateTo(m_baseURL + "pinch-viewport-input-field-long-and-wide.html");
   webViewImpl()->setInitialFocus(false);
   visualViewport.setLocation(FloatPoint());
   frame()->view()->setScrollPosition(IntPoint(0, 0), ProgrammaticScroll);
   webViewImpl()->resizeVisualViewport(IntSize(500, 300));
   visualViewport.setLocation(FloatPoint(30, 50));
 
   webViewImpl()->setPageScaleFactor(2);
   webViewImpl()->scrollFocusedEditableElementIntoRect(IntRect(0, 0, 500, 200));
   EXPECT_POINT_EQ(IntPoint(200 - 30 - 75, 600 - 50 - 65),
                   frame()->view()->scrollPosition());
   EXPECT_FLOAT_POINT_EQ(FloatPoint(30, 50),
                         visualViewport.visibleRect().location());
 }
 
-// Test that resizing the WebView causes ViewportConstrained objects to relayout.
+// Test that resizing the WebView causes ViewportConstrained objects to
+// relayout.
 TEST_P(ParameterizedVisualViewportTest,
        TestWebViewResizeCausesViewportConstrainedLayout) {
   initializeWithDesktopSettings();
   webViewImpl()->resize(IntSize(500, 300));
 
   registerMockedHttpURLLoad("pinch-viewport-fixed-pos.html");
   navigateTo(m_baseURL + "pinch-viewport-fixed-pos.html");
 
   LayoutObject* navbar =
       frame()->document()->getElementById("navbar")->layoutObject();
@@ skipping 53 matching lines @@
 
   // Do a sanity check with no scale applied.
   webViewImpl()->mainFrameImpl()->setClient(&mockWebFrameClient);
   webViewImpl()->handleInputEvent(mouseDownEvent);
   webViewImpl()->handleInputEvent(mouseUpEvent);
 
   Mock::VerifyAndClearExpectations(&mockWebFrameClient);
   mouseDownEvent.button = WebMouseEvent::Button::Left;
   webViewImpl()->handleInputEvent(mouseDownEvent);
 
-  // Now pinch zoom into the page and move the visual viewport. The context
-  // menu should still appear at the location of the event, relative to the
-  // WebView.
+  // Now pinch zoom into the page and move the visual viewport. The context menu
+  // should still appear at the location of the event, relative to the WebView.
   VisualViewport& visualViewport =
       frame()->page()->frameHost().visualViewport();
   webViewImpl()->setPageScaleFactor(2);
   visualViewport.setLocation(FloatPoint(60, 80));
   EXPECT_CALL(mockWebFrameClient, showContextMenu(ContextMenuAtLocation(
                                       mouseDownEvent.x, mouseDownEvent.y)));
 
   mouseDownEvent.button = WebMouseEvent::Button::Right;
   webViewImpl()->handleInputEvent(mouseDownEvent);
   webViewImpl()->handleInputEvent(mouseUpEvent);
@@ skipping 128 matching lines @@
   visualViewport.setScale(1);
   EXPECT_SIZE_EQ(IntSize(500, 450), visualViewport.visibleRect().size());
   EXPECT_SIZE_EQ(IntSize(1000, 900), frameView.frameRect().size());
 
   // Simulate bringing down the top controls by 20px.
   webViewImpl()->applyViewportDeltas(WebFloatSize(), WebFloatSize(),
                                      WebFloatSize(), 1, 1);
   EXPECT_SIZE_EQ(IntSize(500, 430), visualViewport.visibleRect().size());
 
   // Test that the scroll bounds are adjusted appropriately: the visual viewport
-  // should be shrunk by 20px to 430px. The outer viewport was shrunk to maintain the
-  // aspect ratio so it's height is 860px.
+  // should be shrunk by 20px to 430px. The outer viewport was shrunk to
+  // maintain the aspect ratio so it's height is 860px.
   visualViewport.move(FloatPoint(10000, 10000));
   EXPECT_POINT_EQ(FloatPoint(500, 860 - 430), visualViewport.location());
 
   // The outer viewport (FrameView) should be affected as well.
   frameView.scrollBy(IntSize(10000, 10000), UserScroll);
   EXPECT_POINT_EQ(expectedMaxFrameViewScrollOffset(visualViewport, frameView),
                   frameView.scrollPosition());
 
   // Simulate bringing up the top controls by 10.5px.
   webViewImpl()->applyViewportDeltas(WebFloatSize(), WebFloatSize(),
@@ skipping 19 matching lines @@
   navigateTo(m_baseURL + "content-width-1000.html");
 
   VisualViewport& visualViewport =
       frame()->page()->frameHost().visualViewport();
   FrameView& frameView = *webViewImpl()->mainFrameImpl()->frameView();
 
   visualViewport.setScale(2);
   EXPECT_SIZE_EQ(IntSize(250, 225), visualViewport.visibleRect().size());
   EXPECT_SIZE_EQ(IntSize(1000, 900), frameView.frameRect().size());
 
-  // Simulate bringing down the top controls by 20px. Since we're zoomed in,
-  // the top controls take up half as much space (in document-space) than
-  // they do at an unzoomed level.
+  // Simulate bringing down the top controls by 20px. Since we're zoomed in, the
+  // top controls take up half as much space (in document-space) than they do at
+  // an unzoomed level.
   webViewImpl()->applyViewportDeltas(WebFloatSize(), WebFloatSize(),
                                      WebFloatSize(), 1, 1);
   EXPECT_SIZE_EQ(IntSize(250, 215), visualViewport.visibleRect().size());
 
   // Test that the scroll bounds are adjusted appropriately.
   visualViewport.move(FloatPoint(10000, 10000));
   EXPECT_POINT_EQ(FloatPoint(750, 860 - 215), visualViewport.location());
 
   // The outer viewport (FrameView) should be affected as well.
   frameView.scrollBy(IntSize(10000, 10000), UserScroll);
@@ skipping 28 matching lines @@
   visualViewport.move(FloatPoint(10000, 10000));
   EXPECT_FLOAT_POINT_EQ(FloatPoint(375, 877.5 - 548.75),
                         visualViewport.location());
 
   frameView.scrollBy(IntSize(10000, 10000), UserScroll);
   EXPECT_POINT_EQ(expectedMaxFrameViewScrollOffset(visualViewport, frameView),
                   frameView.scrollPosition());
 }
 
 // Tests that a scroll all the way to the bottom of the page, while hiding the
-// top controls doesn't cause a clamp in the viewport scroll offset when the
-// top controls initiated resize occurs.
+// top controls doesn't cause a clamp in the viewport scroll offset when the top
+// controls initiated resize occurs.
 TEST_F(VisualViewportTest, TestTopControlsAdjustmentAndResize) {
   int topControlsHeight = 20;
   int visualViewportHeight = 450;
   int layoutViewportHeight = 900;
   float pageScale = 2;
   float minPageScale = 0.5;
 
   initializeWithAndroidSettings();
 
   // Initialize with top controls showing and shrinking the Blink size.
@@ skipping 129 matching lines @@
 TEST_F(VisualViewportTest, TestTopControlHidingResizeDoesntClampMainFrame) {
   initializeWithAndroidSettings();
   webViewImpl()->resizeWithTopControls(webViewImpl()->size(), 500, false);
   webViewImpl()->applyViewportDeltas(WebFloatSize(), WebFloatSize(),
                                      WebFloatSize(), 1, 1);
   webViewImpl()->resizeWithTopControls(WebSize(1000, 1000), 500, true);
 
   registerMockedHttpURLLoad("content-width-1000.html");
   navigateTo(m_baseURL + "content-width-1000.html");
 
-  // Scroll the FrameView to the bottom of the page but "hide" the top
-  // controls on the compositor side so the max scroll position should account
-  // for the full viewport height.
+  // Scroll the FrameView to the bottom of the page but "hide" the top controls
+  // on the compositor side so the max scroll position should account for the
+  // full viewport height.
   webViewImpl()->applyViewportDeltas(WebFloatSize(), WebFloatSize(),
                                      WebFloatSize(), 1, -1);
   FrameView& frameView = *webViewImpl()->mainFrameImpl()->frameView();
   frameView.setScrollPosition(IntPoint(0, 10000), ProgrammaticScroll);
   EXPECT_EQ(500, frameView.scrollPositionDouble().y());
 
   // Now send the resize, make sure the scroll offset doesn't change.
   webViewImpl()->resizeWithTopControls(WebSize(1000, 1500), 500, false);
   EXPECT_EQ(500, frameView.scrollPositionDouble().y());
 }
@@ skipping 24 matching lines @@
   initializeWithAndroidSettings();
   webViewImpl()->resize(IntSize(100, 150));
   navigateTo("about:blank");
 
   VisualViewport& visualViewport =
       frame()->page()->frameHost().visualViewport();
   EXPECT_TRUE(visualViewport.layerForHorizontalScrollbar()->parent());
   EXPECT_TRUE(visualViewport.layerForVerticalScrollbar()->parent());
 }
 
-// Tests that the layout viewport's scroll layer bounds are updated in a compositing
-// change update. crbug.com/423188.
+// Tests that the layout viewport's scroll layer bounds are updated in a
+// compositing change update. crbug.com/423188.
 TEST_P(ParameterizedVisualViewportTest,
        TestChangingContentSizeAffectsScrollBounds) {
   initializeWithAndroidSettings();
   webViewImpl()->resize(IntSize(100, 150));
 
   registerMockedHttpURLLoad("content-width-1000.html");
   navigateTo(m_baseURL + "content-width-1000.html");
 
   FrameView& frameView = *webViewImpl()->mainFrameImpl()->frameView();
   WebLayer* scrollLayer = frameView.layerForScrolling()->platformLayer();
@@ skipping 86 matching lines @@
 
   // Load page with no main frame scrolling.
   registerMockedHttpURLLoad("200-by-300-viewport.html");
   navigateTo(m_baseURL + "200-by-300-viewport.html");
 
   VisualViewport& visualViewport =
       frame()->page()->frameHost().visualViewport();
   visualViewport.setScale(2);
 
   // Chrome's quirky behavior regarding viewport scrolling means we treat the
-  // body element as the viewport and don't apply scrolling to the HTML
-  // element.
+  // body element as the viewport and don't apply scrolling to the HTML element.
   RuntimeEnabledFeatures::setScrollTopLeftInteropEnabled(false);
 
   LocalDOMWindow* window =
       webViewImpl()->mainFrameImpl()->frame()->localDOMWindow();
   window->scrollTo(100, 150);
   EXPECT_EQ(100, window->scrollX());
   EXPECT_EQ(150, window->scrollY());
   EXPECT_FLOAT_POINT_EQ(FloatPoint(100, 150), visualViewport.location());
 
   HTMLElement* body = toHTMLBodyElement(window->document()->body());
@@ skipping 12 matching lines @@
   EXPECT_FLOAT_POINT_EQ(FloatPoint(50, 130), visualViewport.location());
 
   visualViewport.setLocation(FloatPoint(10, 20));
   EXPECT_EQ(10, body->scrollLeft());
   EXPECT_EQ(20, body->scrollTop());
   EXPECT_EQ(0, documentElement->scrollLeft());
   EXPECT_EQ(0, documentElement->scrollTop());
   EXPECT_EQ(10, window->scrollX());
   EXPECT_EQ(20, window->scrollY());
 
-  // Turning on the standards-compliant viewport scrolling impl should make
-  // the document element the viewport and not body.
+  // Turning on the standards-compliant viewport scrolling impl should make the
+  // document element the viewport and not body.
   RuntimeEnabledFeatures::setScrollTopLeftInteropEnabled(true);
 
   window->scrollTo(100, 150);
   EXPECT_EQ(100, window->scrollX());
   EXPECT_EQ(150, window->scrollY());
   EXPECT_FLOAT_POINT_EQ(FloatPoint(100, 150), visualViewport.location());
 
   body->setScrollLeft(50);
   body->setScrollTop(130);
   EXPECT_EQ(0, body->scrollLeft());
   EXPECT_EQ(0, body->scrollTop());
   EXPECT_FLOAT_POINT_EQ(FloatPoint(100, 150), visualViewport.location());
 
   documentElement->setScrollLeft(40);
   documentElement->setScrollTop(50);
   EXPECT_EQ(40, documentElement->scrollLeft());
   EXPECT_EQ(50, documentElement->scrollTop());
   EXPECT_FLOAT_POINT_EQ(FloatPoint(40, 50), visualViewport.location());
 
   visualViewport.setLocation(FloatPoint(10, 20));
   EXPECT_EQ(0, body->scrollLeft());
   EXPECT_EQ(0, body->scrollTop());
   EXPECT_EQ(10, documentElement->scrollLeft());
   EXPECT_EQ(20, documentElement->scrollTop());
   EXPECT_EQ(10, window->scrollX());
   EXPECT_EQ(20, window->scrollY());
 }
 
-// Tests that when a new frame is created, it is created with the intended
-// size (i.e. viewport at minimum scale, 100x200 / 0.5).
+// Tests that when a new frame is created, it is created with the intended size
+// (i.e. viewport at minimum scale, 100x200 / 0.5).
 TEST_P(ParameterizedVisualViewportTest, TestMainFrameInitializationSizing) {
   initializeWithAndroidSettings();
 
   webViewImpl()->resize(IntSize(100, 200));
 
   registerMockedHttpURLLoad("content-width-1000-min-scale.html");
   navigateTo(m_baseURL + "content-width-1000-min-scale.html");
 
   WebLocalFrameImpl* localFrame = webViewImpl()->mainFrameImpl();
-  // The shutdown() calls are a hack to prevent this test
-  // from violating invariants about frame state during navigation/detach.
+  // The shutdown() calls are a hack to prevent this test from violating
+  // invariants about frame state during navigation/detach.
   localFrame->frame()->document()->shutdown();
   localFrame->createFrameView();
 
   FrameView& frameView = *localFrame->frameView();
   EXPECT_SIZE_EQ(IntSize(200, 400), frameView.frameRect().size());
   frameView.dispose();
 }
 
 // Tests that the maximum scroll offset of the viewport can be fractional.
 TEST_P(ParameterizedVisualViewportTest, FractionalMaxScrollOffset) {
   initializeWithDesktopSettings();
   webViewImpl()->resize(IntSize(101, 201));
   navigateTo("about:blank");
 
   VisualViewport& visualViewport =
       frame()->page()->frameHost().visualViewport();
   ScrollableArea* scrollableArea = &visualViewport;
 
   webViewImpl()->setPageScaleFactor(1.0);
   EXPECT_FLOAT_POINT_EQ(DoublePoint(),
                         scrollableArea->maximumScrollPositionDouble());
 
   webViewImpl()->setPageScaleFactor(2);
   EXPECT_FLOAT_POINT_EQ(DoublePoint(101. / 2., 201. / 2.),
                         scrollableArea->maximumScrollPositionDouble());
 }
 
-// Tests that the slow scrolling after an impl scroll on the visual viewport
-// is continuous. crbug.com/453460 was caused by the impl-path not updating the
+// Tests that the slow scrolling after an impl scroll on the visual viewport is
+// continuous. crbug.com/453460 was caused by the impl-path not updating the
 // ScrollAnimatorBase class.
 TEST_P(ParameterizedVisualViewportTest, SlowScrollAfterImplScroll) {
   initializeWithDesktopSettings();
   webViewImpl()->resize(IntSize(800, 600));
   navigateTo("about:blank");
 
   VisualViewport& visualViewport =
       frame()->page()->frameHost().visualViewport();
 
   // Apply some scroll and scale from the impl-side.
@@ skipping 35 matching lines @@
   FrameView& frameView = *webViewImpl()->mainFrameImpl()->frameView();
 
   webViewImpl()->setPageScaleFactor(2);
   webViewImpl()->setVisualViewportOffset(WebFloatPoint(200, 230));
   frameView.layoutViewportScrollableArea()->setScrollPosition(
       DoublePoint(400, 1100), ProgrammaticScroll);
 
   // FIXME(504057): PaintLayerScrollableArea dirties the compositing state.
   forceFullCompositingUpdate();
 
-  // Because of where the visual viewport is located, this should hit the bottom right
-  // target (target 4).
+  // Because of where the visual viewport is located, this should hit the bottom
+  // right target (target 4).
   WebAXObject hitNode =
       webDoc.accessibilityObject().hitTest(WebPoint(154, 165));
   WebAXNameFrom nameFrom;
   WebVector<WebAXObject> nameObjects;
   EXPECT_EQ(std::string("Target4"), hitNode.name(nameFrom, nameObjects).utf8());
 }
 
 // Tests that the maximum scroll offset of the viewport can be fractional.
 TEST_P(ParameterizedVisualViewportTest, TestCoordinateTransforms) {
   initializeWithAndroidSettings();
@@ skipping 188 matching lines @@
 
   EXPECT_POINT_EQ(
       DoublePoint(),
       frameView.layoutViewportScrollableArea()->scrollPositionDouble());
   EXPECT_EQ(600, scroller->scrollTop());
 
   RuntimeEnabledFeatures::setSetRootScrollerEnabled(wasRootScrollerEnabled);
 }
 
 }  // namespace