reflow comments in core/layout/[M-Z]*.{cpp,h}

third_party/WebKit/Source/core/layout/VisualRectMappingTest.cpp

 // Copyright 2015 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/layout/LayoutTestHelper.h"
 #include "core/layout/LayoutView.h"
 #include "core/layout/PaintInvalidationState.h"
 #include "core/paint/PaintLayer.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
@@ skipping 121 matching lines @@
       "frame",
       "<style>body { margin: 0; }</style><span><img style='width: 20px; "
       "height: 100px'></span>text text text");
   document().view()->updateAllLifecyclePhases();
 
   LayoutBlock* frameContainer =
       toLayoutBlock(getLayoutObjectByElementId("frameContainer"));
   LayoutBlock* frameBody = toLayoutBlock(frameDocument.body()->layoutObject());
   LayoutText* frameText = toLayoutText(frameBody->lastChild());
 
-  // This case involves clipping: frame height is 50, y-coordinate of result rect is 13,
-  // so height should be clipped to (50 - 13) == 37.
+  // This case involves clipping: frame height is 50, y-coordinate of result
+  // rect is 13, so height should be clipped to (50 - 13) == 37.
   frameDocument.view()->setScrollOffset(ScrollOffset(0, 47),
                                         ProgrammaticScroll);
   LayoutRect originalRect(4, 60, 20, 80);
   LayoutRect rect = originalRect;
   EXPECT_TRUE(frameText->mapToVisualRectInAncestorSpace(frameContainer, rect));
   EXPECT_EQ(rect, LayoutRect(4, 13, 20, 37));
 
   rect = originalRect;
   EXPECT_TRUE(frameText->mapToVisualRectInAncestorSpace(&layoutView(), rect));
   EXPECT_EQ(rect, LayoutRect(4, 13, 20, 37));
@@ skipping 19 matching lines @@
       "frame",
       "<style>body { margin: 0; }</style><div id='target' style='position: "
       "relative; width: 100px; height: 100px; left: 0.5px'>");
   document().view()->updateAllLifecyclePhases();
 
   LayoutBlock* frameContainer =
       toLayoutBlock(getLayoutObjectByElementId("frameContainer"));
   LayoutObject* target = frameDocument.getElementById("target")->layoutObject();
   LayoutRect rect(0, 0, 100, 100);
   EXPECT_TRUE(target->mapToVisualRectInAncestorSpace(frameContainer, rect));
-  // When passing from the iframe to the parent frame, the rect of (0.5, 0, 100, 100) is expanded to (0, 0, 100, 100), and then offset by
-  // the 0.5 offset of frameContainer.
+  // When passing from the iframe to the parent frame, the rect of (0.5, 0, 100,
+  // 100) is expanded to (0, 0, 100, 100), and then offset by the 0.5 offset of
+  // frameContainer.
   EXPECT_EQ(LayoutRect(LayoutPoint(DoublePoint(0.5, 0)), LayoutSize(101, 100)),
             rect);
 }
 
 TEST_F(VisualRectMappingTest, LayoutViewDisplayNone) {
   document().setBaseURLOverride(KURL(ParsedURLString, "http://test.com"));
   setBodyInnerHTML(
       "<style>body { margin: 0; }</style>"
       "<div id=frameContainer>"
       "  <iframe id=frame src='http://test.com' width='50' height='50' "
       "frameBorder='0'></iframe>"
       "</div>");
 
   Document& frameDocument =
       setupChildIframe("frame",
                        "<style>body { margin: 0; }</style><div "
                        "style='width:100px;height:100px;'></div>");
   document().view()->updateAllLifecyclePhases();
 
   LayoutBlock* frameContainer =
       toLayoutBlock(getLayoutObjectByElementId("frameContainer"));
   LayoutBlock* frameBody = toLayoutBlock(frameDocument.body()->layoutObject());
   LayoutBlock* frameDiv = toLayoutBlock(frameBody->lastChild());
 
-  // This part is copied from the LayoutView test, just to ensure that the mapped
-  // rect is valid before display:none is set on the iframe.
+  // This part is copied from the LayoutView test, just to ensure that the
+  // mapped rect is valid before display:none is set on the iframe.
   frameDocument.view()->setScrollOffset(ScrollOffset(0, 47),
                                         ProgrammaticScroll);
   LayoutRect originalRect(4, 60, 20, 80);
   LayoutRect rect = originalRect;
   EXPECT_TRUE(frameDiv->mapToVisualRectInAncestorSpace(frameContainer, rect));
   EXPECT_EQ(rect, LayoutRect(4, 13, 20, 37));
 
   Element* frameElement = document().getElementById("frame");
   frameElement->setInlineStyleProperty(CSSPropertyDisplay, "none");
   document().view()->updateAllLifecyclePhases();
 
   rect = originalRect;
   EXPECT_FALSE(frameDiv->mapToVisualRectInAncestorSpace(&layoutView(), rect));
   EXPECT_EQ(rect, LayoutRect());
 }
 
 TEST_F(VisualRectMappingTest, SelfFlippedWritingMode) {
   setBodyInnerHTML(
       "<div id='target' style='writing-mode: vertical-rl; box-shadow: 40px "
       "20px black;"
       "    width: 100px; height: 50px; position: absolute; top: 111px; left: "
       "222px'>"
       "</div>");
 
   LayoutBlock* target = toLayoutBlock(getLayoutObjectByElementId("target"));
   LayoutRect overflowRect = target->localOverflowRectForPaintInvalidation();
-  // -40 = -box_shadow_offset_x(40) (with target's top-right corner as the origin)
+  // -40 = -box_shadow_offset_x(40) (with target's top-right corner as the
+  // origin)
   // 140 = width(100) + box_shadow_offset_x(40)
   // 70 = height(50) + box_shadow_offset_y(20)
   EXPECT_EQ(LayoutRect(-40, 0, 140, 70), overflowRect);
 
   LayoutRect rect = overflowRect;
   // TODO(wkorman): The calls to flipForWritingMode() here and in other test
   // cases below are necessary because mapToVisualRectInAncestorSpace()
   // currently expects the input rect to be in "physical coordinates" (*not*
   // "physical coordinates with flipped block-flow direction"), see
   // LayoutBoxModelObject.h.
@@ skipping 15 matching lines @@
       "<div id='container' style='writing-mode: vertical-rl; position: "
       "absolute; top: 111px; left: 222px'>"
       "    <div id='target' style='box-shadow: 40px 20px black; width: 100px; "
       "height: 90px'></div>"
       "    <div style='width: 100px; height: 100px'></div>"
       "</div>");
 
   LayoutBlock* target = toLayoutBlock(getLayoutObjectByElementId("target"));
   LayoutRect targetOverflowRect =
       target->localOverflowRectForPaintInvalidation();
-  // -40 = -box_shadow_offset_x(40) (with target's top-right corner as the origin)
+  // -40 = -box_shadow_offset_x(40) (with target's top-right corner as the
+  // origin)
   // 140 = width(100) + box_shadow_offset_x(40)
   // 110 = height(90) + box_shadow_offset_y(20)
   EXPECT_EQ(LayoutRect(-40, 0, 140, 110), targetOverflowRect);
 
   LayoutRect rect = targetOverflowRect;
   target->flipForWritingMode(rect);
   EXPECT_TRUE(target->mapToVisualRectInAncestorSpace(target, rect));
   // This rect is in physical coordinates of target.
   EXPECT_EQ(LayoutRect(0, 0, 140, 110), rect);
 
@@ skipping 51 matching lines @@
   EXPECT_EQ(LayoutRect(0, 0, 140, 110), targetOverflowRect);
   LayoutRect rect = targetOverflowRect;
   EXPECT_TRUE(target->mapToVisualRectInAncestorSpace(target, rect));
   EXPECT_EQ(LayoutRect(0, 0, 140, 110), rect);
 
   rect = targetOverflowRect;
   EXPECT_TRUE(target->mapToVisualRectInAncestorSpace(container, rect));
   rect.move(-container->scrolledContentOffset());
   // 2 = target_x(0) + container_border_left(10) - scroll_left(8)
   // 3 = target_y(0) + container_border_top(10) - scroll_top(7)
-  // Rect is not clipped by container's overflow clip because of overflow:scroll.
+  // Rect is not clipped by container's overflow clip because of
+  // overflow:scroll.
   EXPECT_EQ(LayoutRect(2, 3, 140, 110), rect);
 
   rect = targetOverflowRect;
   EXPECT_TRUE(target->mapToVisualRectInAncestorSpace(&layoutView(), rect));
-  // (2, 3, 140, 100) is first clipped by container's overflow clip, to (10, 10, 50, 80),
-  // then is by added container's offset in LayoutView (111, 222).
+  // (2, 3, 140, 100) is first clipped by container's overflow clip, to
+  // (10, 10, 50, 80), then is by added container's offset in LayoutView
+  // (111, 222).
   EXPECT_EQ(LayoutRect(232, 121, 50, 80), rect);
   checkPaintInvalidationStateRectMapping(rect, targetOverflowRect, *target,
                                          layoutView(), layoutView());
 
   LayoutRect containerOverflowRect =
       container->localOverflowRectForPaintInvalidation();
-  // Because container has overflow clip, its visual overflow doesn't include overflow from children.
+  // Because container has overflow clip, its visual overflow doesn't include
+  // overflow from children.
   // 70 = width(50) + border_left_width(10) + border_right_width(10)
   // 100 = height(80) + border_top_width(10) + border_bottom_width(10)
   EXPECT_EQ(LayoutRect(0, 0, 70, 100), containerOverflowRect);
   rect = containerOverflowRect;
   EXPECT_TRUE(container->mapToVisualRectInAncestorSpace(container, rect));
   // Container should not apply overflow clip on its own overflow rect.
   EXPECT_EQ(LayoutRect(0, 0, 70, 100), rect);
 
   rect = containerOverflowRect;
   EXPECT_TRUE(container->mapToVisualRectInAncestorSpace(&layoutView(), rect));
   EXPECT_EQ(LayoutRect(222, 111, 70, 100), rect);
   checkPaintInvalidationStateRectMapping(
       rect, containerOverflowRect, *container, layoutView(), layoutView());
 }
 
 TEST_F(VisualRectMappingTest, ContainerFlippedWritingModeAndOverflowScroll) {
   setBodyInnerHTML(
       "<div id='container' style='writing-mode: vertical-rl; position: "
       "absolute; top: 111px; left: 222px;"
       "    border: solid red; border-width: 10px 20px 30px 40px;"
       "    overflow: scroll; width: 50px; height: 80px'>"
       "    <div id='target' style='box-shadow: 40px 20px black; width: 100px; "
       "height: 90px'></div>"
       "    <div style='width: 100px; height: 100px'></div>"
       "</div>");
 
   LayoutBlock* container =
       toLayoutBlock(getLayoutObjectByElementId("container"));
   EXPECT_EQ(LayoutUnit(), container->scrollTop());
-  // The initial scroll offset is to the left-most because of flipped blocks writing mode.
+  // The initial scroll offset is to the left-most because of flipped blocks
+  // writing mode.
   // 150 = total_layout_overflow(100 + 100) - width(50)
   EXPECT_EQ(LayoutUnit(150), container->scrollLeft());
   container->setScrollTop(LayoutUnit(7));
   container->setScrollLeft(
       LayoutUnit(142));  // Scroll to the right by 8 pixels.
   document().view()->updateAllLifecyclePhases();
 
   LayoutBlock* target = toLayoutBlock(getLayoutObjectByElementId("target"));
   LayoutRect targetOverflowRect =
       target->localOverflowRectForPaintInvalidation();
-  // -40 = -box_shadow_offset_x(40) (with target's top-right corner as the origin)
+  // -40 = -box_shadow_offset_x(40) (with target's top-right corner as the
+  // origin)
   // 140 = width(100) + box_shadow_offset_x(40)
   // 110 = height(90) + box_shadow_offset_y(20)
   EXPECT_EQ(LayoutRect(-40, 0, 140, 110), targetOverflowRect);
 
   LayoutRect rect = targetOverflowRect;
   target->flipForWritingMode(rect);
   EXPECT_TRUE(target->mapToVisualRectInAncestorSpace(target, rect));
   // This rect is in physical coordinates of target.
   EXPECT_EQ(LayoutRect(0, 0, 140, 110), rect);
 
   rect = targetOverflowRect;
   target->flipForWritingMode(rect);
   EXPECT_TRUE(target->mapToVisualRectInAncestorSpace(container, rect));
   rect.move(-container->scrolledContentOffset());
   // -2 = target_physical_x(100) + container_border_left(40) - scroll_left(142)
   // 3 = target_y(0) + container_border_top(10) - scroll_top(7)
   // Rect is clipped by container's overflow clip because of overflow:scroll.
   EXPECT_EQ(LayoutRect(-2, 3, 140, 110), rect);
 
   rect = targetOverflowRect;
   target->flipForWritingMode(rect);
   EXPECT_TRUE(target->mapToVisualRectInAncestorSpace(&layoutView(), rect));
-  // (-2, 3, 140, 100) is first clipped by container's overflow clip, to (40, 10, 50, 80),
-  // then is added by container's offset in LayoutView (111, 222).
-  // TODO(crbug.com/600039): rect.x() should be 262 (left + border-left), but is offset
+  // (-2, 3, 140, 100) is first clipped by container's overflow clip, to
+  // (40, 10, 50, 80), then is added by container's offset in LayoutView
+  // (111, 222).
+  // TODO(crbug.com/600039): rect.x() should be 262 (left + border-left), but is
+  // offset
   // by extra horizontal border-widths because of layout error.
   EXPECT_EQ(LayoutRect(322, 121, 50, 80), rect);
   checkPaintInvalidationStateRectMapping(rect, targetOverflowRect, *target,
                                          layoutView(), layoutView());
 
   LayoutRect containerOverflowRect =
       container->localOverflowRectForPaintInvalidation();
-  // Because container has overflow clip, its visual overflow doesn't include overflow from children.
+  // Because container has overflow clip, its visual overflow doesn't include
+  // overflow from children.
   // 110 = width(50) + border_left_width(40) + border_right_width(20)
   // 120 = height(80) + border_top_width(10) + border_bottom_width(30)
   EXPECT_EQ(LayoutRect(0, 0, 110, 120), containerOverflowRect);
 
   rect = containerOverflowRect;
   container->flipForWritingMode(rect);
   EXPECT_TRUE(container->mapToVisualRectInAncestorSpace(container, rect));
   EXPECT_EQ(LayoutRect(0, 0, 110, 120), rect);
 
   rect = containerOverflowRect;
   container->flipForWritingMode(rect);
   EXPECT_TRUE(container->mapToVisualRectInAncestorSpace(&layoutView(), rect));
-  // TODO(crbug.com/600039): rect.x() should be 222 (left), but is offset by extra horizontal
+  // TODO(crbug.com/600039): rect.x() should be 222 (left), but is offset by
+  // extra horizontal
   // border-widths because of layout error.
   EXPECT_EQ(LayoutRect(282, 111, 110, 120), rect);
   checkPaintInvalidationStateRectMapping(
       rect, containerOverflowRect, *container, layoutView(), layoutView());
 }
 
 TEST_F(VisualRectMappingTest, ContainerOverflowHidden) {
   setBodyInnerHTML(
       "<div id='container' style='position: absolute; top: 111px; left: 222px;"
       "    border: 10px solid red; overflow: hidden; width: 50px; height: "
@@ skipping 33 matching lines @@
       "    border: solid red; border-width: 10px 20px 30px 40px;"
       "    overflow: hidden; width: 50px; height: 80px'>"
       "    <div id='target' style='box-shadow: 40px 20px black; width: 100px; "
       "height: 90px'></div>"
       "    <div style='width: 100px; height: 100px'></div>"
       "</div>");
 
   LayoutBlock* container =
       toLayoutBlock(getLayoutObjectByElementId("container"));
   EXPECT_EQ(LayoutUnit(), container->scrollTop());
-  // The initial scroll offset is to the left-most because of flipped blocks writing mode.
+  // The initial scroll offset is to the left-most because of flipped blocks
+  // writing mode.
   // 150 = total_layout_overflow(100 + 100) - width(50)
   EXPECT_EQ(LayoutUnit(150), container->scrollLeft());
   container->setScrollTop(LayoutUnit(7));
   container->setScrollLeft(LayoutUnit(82));  // Scroll to the right by 8 pixels.
   document().view()->updateAllLifecyclePhases();
 
   LayoutBlock* target = toLayoutBlock(getLayoutObjectByElementId("target"));
   LayoutRect targetOverflowRect =
       target->localOverflowRectForPaintInvalidation();
-  // -40 = -box_shadow_offset_x(40) (with target's top-right corner as the origin)
+  // -40 = -box_shadow_offset_x(40) (with target's top-right corner as the
+  // origin)
   // 140 = width(100) + box_shadow_offset_x(40)
   // 110 = height(90) + box_shadow_offset_y(20)
   EXPECT_EQ(LayoutRect(-40, 0, 140, 110), targetOverflowRect);
 
   LayoutRect rect = targetOverflowRect;
   target->flipForWritingMode(rect);
   EXPECT_TRUE(target->mapToVisualRectInAncestorSpace(target, rect));
   // This rect is in physical coordinates of target.
   EXPECT_EQ(LayoutRect(0, 0, 140, 110), rect);
 
@@ skipping 11 matching lines @@
       "    border: solid red; border-width: 10px 20px 30px 40px;"
       "    overflow: scroll; width: 50px; height: 80px'>"
       "    <div id='target' style='writing-mode: vertical-lr; box-shadow: 40px "
       "20px black; width: 100px; height: 90px'></div>"
       "    <div style='width: 100px; height: 100px'></div>"
       "</div>");
 
   LayoutBlock* container =
       toLayoutBlock(getLayoutObjectByElementId("container"));
   EXPECT_EQ(LayoutUnit(), container->scrollTop());
-  // The initial scroll offset is to the left-most because of flipped blocks writing mode.
+  // The initial scroll offset is to the left-most because of flipped blocks
+  // writing mode.
   // 150 = total_layout_overflow(100 + 100) - width(50)
   EXPECT_EQ(LayoutUnit(150), container->scrollLeft());
   container->setScrollTop(LayoutUnit(7));
   container->setScrollLeft(
       LayoutUnit(142));  // Scroll to the right by 8 pixels.
   document().view()->updateAllLifecyclePhases();
 
   LayoutBlock* target = toLayoutBlock(getLayoutObjectByElementId("target"));
   LayoutRect targetOverflowRect =
       target->localOverflowRectForPaintInvalidation();
@@ skipping 70 matching lines @@
        ContainerOfAbsoluteAbovePaintInvalidationContainer) {
   enableCompositing();
   document().frame()->settings()->setPreferCompositingToLCDTextEnabled(true);
 
   setBodyInnerHTML(
       "<div id='container' style='position: absolute; top: 88px; left: 99px'>"
       "    <div style='height: 222px'></div>"
       // This div makes stacking-context composited.
       "    <div style='position: absolute; width: 1px; height: 1px; "
       "background:yellow; will-change: transform'></div>"
-      // This stacking context is paintInvalidationContainer of the absolute child, but not a container of it.
+      // This stacking context is paintInvalidationContainer of the absolute
+      // child, but not a container of it.
       "    <div id='stacking-context' style='opacity: 0.9'>"
       "        <div id='absolute' style='position: absolute; top: 50px; left: "
       "50px; width: 50px; height: 50px; background: green'></div>"
       "    </div>"
       "</div>");
 
   LayoutBlock* stackingContext =
       toLayoutBlock(getLayoutObjectByElementId("stacking-context"));
   LayoutBlock* absolute = toLayoutBlock(getLayoutObjectByElementId("absolute"));
   LayoutBlock* container =
@@ skipping 46 matching lines @@
       target->localOverflowRectForPaintInvalidation();
   EXPECT_EQ(LayoutRect(0, 0, 400, 400), targetOverflowRect);
   LayoutRect rect = targetOverflowRect;
   EXPECT_TRUE(target->mapToVisualRectInAncestorSpace(&layoutView(), rect));
   EXPECT_EQ(LayoutRect(0, 0, 200, 200), rect);
   checkPaintInvalidationStateRectMapping(rect, targetOverflowRect, *target,
                                          layoutView(), layoutView());
 }
 
 }  // namespace blink