reflow comments in core/paint

third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilderTest.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/LayoutTreeAsText.h"
 #include "core/layout/api/LayoutViewItem.h"
 #include "core/paint/ObjectPaintProperties.h"
 #include "core/paint/PaintPropertyTreePrinter.h"
 #include "platform/graphics/paint/GeometryMapper.h"
@@ skipping 134 matching lines @@
   loadTestData("fixed-position.html");
 
   Element* positionedScroll = document().getElementById("positionedScroll");
   positionedScroll->setScrollTop(3);
   Element* transformedScroll = document().getElementById("transformedScroll");
   transformedScroll->setScrollTop(5);
 
   FrameView* frameView = document().view();
   frameView->updateAllLifecyclePhases();
 
-  // target1 is a fixed-position element inside an absolute-position scrolling element.
-  // It should be attached under the viewport to skip scrolling and offset of the parent.
+  // target1 is a fixed-position element inside an absolute-position scrolling
+  // element.  It should be attached under the viewport to skip scrolling and
+  // offset of the parent.
   Element* target1 = document().getElementById("target1");
   const ObjectPaintProperties* target1Properties =
       target1->layoutObject()->objectPaintProperties();
   EXPECT_EQ(TransformationMatrix().translate(200, 150),
             target1Properties->paintOffsetTranslation()->matrix());
   EXPECT_EQ(framePreTranslation(),
             target1Properties->paintOffsetTranslation()->parent());
   EXPECT_EQ(target1Properties->paintOffsetTranslation(),
             target1Properties->overflowClip()->localTransformSpace());
   EXPECT_EQ(FloatRoundedRect(0, 0, 100, 100),
             target1Properties->overflowClip()->clipRect());
-  // Likewise, it inherits clip from the viewport, skipping overflow clip of the scroller.
+  // Likewise, it inherits clip from the viewport, skipping overflow clip of the
+  // scroller.
   EXPECT_EQ(frameContentClip(), target1Properties->overflowClip()->parent());
-  // target1 should not have it's own scroll node and instead should inherit positionedScroll's.
+  // target1 should not have it's own scroll node and instead should inherit
+  // positionedScroll's.
   const ObjectPaintProperties* positionedScrollProperties =
       positionedScroll->layoutObject()->objectPaintProperties();
   EXPECT_TRUE(positionedScrollProperties->scroll()->parent()->isRoot());
   EXPECT_EQ(TransformationMatrix().translate(0, -3),
             positionedScrollProperties->scroll()
                 ->scrollOffsetTranslation()
                 ->matrix());
   EXPECT_EQ(nullptr, target1Properties->scroll());
 
   CHECK_EXACT_VISUAL_RECT(LayoutRect(200, 150, 100, 100),
@@ skipping 10 matching lines @@
   EXPECT_EQ(TransformationMatrix().translate(200, 150),
             target2Properties->paintOffsetTranslation()->matrix());
   EXPECT_EQ(scrollerProperties->scrollTranslation(),
             target2Properties->paintOffsetTranslation()->parent());
   EXPECT_EQ(target2Properties->paintOffsetTranslation(),
             target2Properties->overflowClip()->localTransformSpace());
   EXPECT_EQ(FloatRoundedRect(0, 0, 100, 100),
             target2Properties->overflowClip()->clipRect());
   EXPECT_EQ(scrollerProperties->overflowClip(),
             target2Properties->overflowClip()->parent());
-  // target2 should not have it's own scroll node and instead should inherit transformedScroll's.
+  // target2 should not have it's own scroll node and instead should inherit
+  // transformedScroll's.
   const ObjectPaintProperties* transformedScrollProperties =
       transformedScroll->layoutObject()->objectPaintProperties();
   EXPECT_TRUE(transformedScrollProperties->scroll()->parent()->isRoot());
   EXPECT_EQ(TransformationMatrix().translate(0, -5),
             transformedScrollProperties->scroll()
                 ->scrollOffsetTranslation()
                 ->matrix());
   EXPECT_EQ(nullptr, target2Properties->scroll());
 
   CHECK_EXACT_VISUAL_RECT(LayoutRect(208, 153, 200, 100),
@@ skipping 22 matching lines @@
   EXPECT_TRUE(scrollerProperties->scroll()->userScrollableVertical());
   EXPECT_EQ(FloatRoundedRect(120, 340, 413, 317),
             scrollerProperties->overflowClip()->clipRect());
   EXPECT_EQ(frameContentClip(), scrollerProperties->overflowClip()->parent());
   // http://crbug.com/638415
   if (!RuntimeEnabledFeatures::rootLayerScrollingEnabled()) {
     CHECK_EXACT_VISUAL_RECT(LayoutRect(120, 340, 413, 317),
                             scroller->layoutObject(), frameView->layoutView());
   }
 
-  // The relative-positioned element should have accumulated box offset (exclude scrolling),
-  // and should be affected by ancestor scroll transforms.
+  // The relative-positioned element should have accumulated box offset (exclude
+  // scrolling), and should be affected by ancestor scroll transforms.
   Element* relPos = document().getElementById("rel-pos");
   const ObjectPaintProperties* relPosProperties =
       relPos->layoutObject()->objectPaintProperties();
   EXPECT_EQ(TransformationMatrix().translate(680, 1120),
             relPosProperties->paintOffsetTranslation()->matrix());
   EXPECT_EQ(scrollerProperties->scrollTranslation(),
             relPosProperties->paintOffsetTranslation()->parent());
   EXPECT_EQ(relPosProperties->transform(),
             relPosProperties->overflowClip()->localTransformSpace());
   EXPECT_EQ(FloatRoundedRect(0, 0, 100, 200),
             relPosProperties->overflowClip()->clipRect());
   EXPECT_EQ(scrollerProperties->overflowClip(),
             relPosProperties->overflowClip()->parent());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(), relPos->layoutObject(),
                           frameView->layoutView());
 
-  // The absolute-positioned element should not be affected by non-positioned scroller at all.
+  // The absolute-positioned element should not be affected by non-positioned
+  // scroller at all.
   Element* absPos = document().getElementById("abs-pos");
   const ObjectPaintProperties* absPosProperties =
       absPos->layoutObject()->objectPaintProperties();
   EXPECT_EQ(TransformationMatrix().translate(123, 456),
             absPosProperties->paintOffsetTranslation()->matrix());
   EXPECT_EQ(frameScrollTranslation(),
             absPosProperties->paintOffsetTranslation()->parent());
   EXPECT_EQ(absPosProperties->transform(),
             absPosProperties->overflowClip()->localTransformSpace());
   EXPECT_EQ(FloatRoundedRect(0, 0, 300, 400),
@@ skipping 36 matching lines @@
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, Perspective) {
   loadTestData("perspective.html");
 
   Element* perspective = document().getElementById("perspective");
   const ObjectPaintProperties* perspectiveProperties =
       perspective->layoutObject()->objectPaintProperties();
   EXPECT_EQ(TransformationMatrix().applyPerspective(100),
             perspectiveProperties->perspective()->matrix());
-  // The perspective origin is the center of the border box plus accumulated paint offset.
+  // The perspective origin is the center of the border box plus accumulated
+  // paint offset.
   EXPECT_EQ(FloatPoint3D(250, 250, 0),
             perspectiveProperties->perspective()->origin());
   EXPECT_EQ(framePreTranslation(),
             perspectiveProperties->perspective()->parent());
 
-  // Adding perspective doesn't clear paint offset. The paint offset will be passed down to children.
+  // Adding perspective doesn't clear paint offset. The paint offset will be
+  // passed down to children.
   Element* inner = document().getElementById("inner");
   const ObjectPaintProperties* innerProperties =
       inner->layoutObject()->objectPaintProperties();
   EXPECT_EQ(TransformationMatrix().translate(50, 100),
             innerProperties->paintOffsetTranslation()->matrix());
   EXPECT_EQ(perspectiveProperties->perspective(),
             innerProperties->paintOffsetTranslation()->parent());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(50, 100, 100, 200), inner->layoutObject(),
                           document().view()->layoutView());
 }
@@ skipping 196 matching lines @@
   EXPECT_EQ(nullptr, rectWithoutOpacityProperties);
 
   LayoutObject& rectWithOpacity =
       *document().getElementById("rectWithOpacity")->layoutObject();
   const ObjectPaintProperties* rectWithOpacityProperties =
       rectWithOpacity.objectPaintProperties();
   EXPECT_EQ(0.4f, rectWithOpacityProperties->effect()->opacity());
   EXPECT_EQ(groupWithOpacityProperties->effect(),
             rectWithOpacityProperties->effect()->parent());
 
-  // Ensure that opacity nodes are created for LayoutSVGText which inherits from LayoutSVGBlock instead of LayoutSVGModelObject.
+  // Ensure that opacity nodes are created for LayoutSVGText which inherits from
+  // LayoutSVGBlock instead of LayoutSVGModelObject.
   LayoutObject& textWithOpacity =
       *document().getElementById("textWithOpacity")->layoutObject();
   const ObjectPaintProperties* textWithOpacityProperties =
       textWithOpacity.objectPaintProperties();
   EXPECT_EQ(0.2f, textWithOpacityProperties->effect()->opacity());
   EXPECT_EQ(groupWithOpacityProperties->effect(),
             textWithOpacityProperties->effect()->parent());
 
-  // Ensure that opacity nodes are created for LayoutSVGTSpan which inherits from LayoutSVGInline instead of LayoutSVGModelObject.
+  // Ensure that opacity nodes are created for LayoutSVGTSpan which inherits
+  // from LayoutSVGInline instead of LayoutSVGModelObject.
   LayoutObject& tspanWithOpacity =
       *document().getElementById("tspanWithOpacity")->layoutObject();
   const ObjectPaintProperties* tspanWithOpacityProperties =
       tspanWithOpacity.objectPaintProperties();
   EXPECT_EQ(0.1f, tspanWithOpacityProperties->effect()->opacity());
   EXPECT_EQ(textWithOpacityProperties->effect(),
             tspanWithOpacityProperties->effect()->parent());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, EffectNodesAcrossHTMLSVGBoundary) {
@@ skipping 188 matching lines @@
             svgProperties->paintOffsetTranslation()->matrix());
   EXPECT_EQ(TransformationMatrix().translate(5, 7),
             svgProperties->transform()->matrix());
   EXPECT_EQ(TransformationMatrix().translate(11, 11).scale(100.0 / 13.0),
             svgProperties->svgLocalToBorderBoxTransform()->matrix());
   EXPECT_EQ(svgProperties->paintOffsetTranslation(),
             svgProperties->transform()->parent());
   EXPECT_EQ(svgProperties->transform(),
             svgProperties->svgLocalToBorderBoxTransform()->parent());
 
-  // Ensure the rect's transform is a child of the local to border box transform.
+  // Ensure the rect's transform is a child of the local to border box
+  // transform.
   LayoutObject& rect = *document().getElementById("rect")->layoutObject();
   const ObjectPaintProperties* rectProperties = rect.objectPaintProperties();
   EXPECT_EQ(TransformationMatrix().translate(17, 19),
             rectProperties->transform()->matrix());
   EXPECT_EQ(svgProperties->svgLocalToBorderBoxTransform(),
             rectProperties->transform()->parent());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, SVGNestedViewboxTransforms) {
   setBodyInnerHTML(
@@ skipping 88 matching lines @@
   EXPECT_EQ(TransformationMatrix().translate(20, 30),
             containerProperties->transform()->matrix());
   EXPECT_EQ(svgProperties->transform(),
             containerProperties->transform()->parent());
 
   Element* fixed = document().getElementById("fixed");
   const ObjectPaintProperties* fixedProperties =
       fixed->layoutObject()->objectPaintProperties();
   EXPECT_EQ(TransformationMatrix().translate(200, 150),
             fixedProperties->paintOffsetTranslation()->matrix());
-  // Ensure the fixed position element is rooted at the nearest transform container.
+  // Ensure the fixed position element is rooted at the nearest transform
+  // container.
   EXPECT_EQ(containerProperties->transform(),
             fixedProperties->paintOffsetTranslation()->parent());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, ControlClip) {
   setBodyInnerHTML(
       "<style>"
       "  body {"
       "    margin: 0;"
       "  }"
       "  input {"
       "    border-width: 5px;"
       "    padding: 0;"
       "  }"
       "</style>"
       "<input id='button' type='button' style='width:345px; height:123px' "
       "value='some text'/>");
 
   LayoutObject& button = *document().getElementById("button")->layoutObject();
   const ObjectPaintProperties* buttonProperties =
       button.objectPaintProperties();
-  // No scroll translation because the document does not scroll (not enough content).
+  // No scroll translation because the document does not scroll (not enough
+  // content).
   EXPECT_TRUE(!frameScrollTranslation());
   EXPECT_EQ(framePreTranslation(),
             buttonProperties->overflowClip()->localTransformSpace());
   EXPECT_EQ(FloatRoundedRect(5, 5, 335, 113),
             buttonProperties->overflowClip()->clipRect());
   EXPECT_EQ(frameContentClip(), buttonProperties->overflowClip()->parent());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(0, 0, 345, 123), &button,
                           document().view()->layoutView());
 }
 
@@ skipping 11 matching lines @@
       "   border-left: 60px solid;"
       "   width: 500px;"
       "   height: 400px;"
       "   overflow: scroll;"
       " }"
       "</style>"
       "<div id='div'></div>");
 
   LayoutObject& div = *document().getElementById("div")->layoutObject();
   const ObjectPaintProperties* divProperties = div.objectPaintProperties();
-  // No scroll translation because the document does not scroll (not enough content).
+  // No scroll translation because the document does not scroll (not enough
+  // content).
   EXPECT_TRUE(!frameScrollTranslation());
   EXPECT_EQ(framePreTranslation(),
             divProperties->overflowClip()->localTransformSpace());
   // The overflow clip rect includes only the padding box.
-  // padding box = border box(500+60+50, 400+45+55) - border outset(60+50, 45+55) - scrollbars(15, 15)
+  // padding box = border box(500+60+50, 400+45+55) - border outset(60+50,
+  // 45+55) - scrollbars(15, 15)
   EXPECT_EQ(FloatRoundedRect(60, 45, 500, 400),
             divProperties->overflowClip()->clipRect());
   const ClipPaintPropertyNode* borderRadiusClip =
       divProperties->overflowClip()->parent();
   EXPECT_EQ(framePreTranslation(), borderRadiusClip->localTransformSpace());
-  // The border radius clip is the area enclosed by inner border edge, including the scrollbars.
-  // As the border-radius is specified in outer radius, the inner radius is calculated by:
+  // The border radius clip is the area enclosed by inner border edge, including
+  // the scrollbars.  As the border-radius is specified in outer radius, the
+  // inner radius is calculated by:
   // inner radius = max(outer radius - border width, 0)
-  // In the case that two adjacent borders have different width, the inner radius of the corner
-  // may transition from one value to the other. i.e. being an ellipse.
+  // In the case that two adjacent borders have different width, the inner
+  // radius of the corner may transition from one value to the other. i.e. being
+  // an ellipse.
   EXPECT_EQ(
       FloatRoundedRect(
           FloatRect(60, 45, 500,
                     400),  // = border box(610, 500) - border outset(110, 100)
           FloatSize(0,
                     0),  //   (top left)     = max((12, 12) - (60, 45), (0, 0))
           FloatSize(0,
                     0),  //   (top right)    = max((34, 34) - (50, 45), (0, 0))
           FloatSize(18,
                     23),  // (bottom left)  = max((78, 78) - (60, 55), (0, 0))
@@ skipping 34 matching lines @@
   LayoutObject* innerDivWithTransform =
       frameDocument.getElementById("transform")->layoutObject();
   const ObjectPaintProperties* innerDivWithTransformProperties =
       innerDivWithTransform->objectPaintProperties();
   auto* innerDivTransform = innerDivWithTransformProperties->transform();
   EXPECT_EQ(TransformationMatrix().translate3d(4, 5, 6),
             innerDivTransform->matrix());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(12, 14, 100, 145), innerDivWithTransform,
                           frameView->layoutView());
 
-  // Ensure that the inner div's transform is correctly rooted in the root frame's transform tree.
+  // Ensure that the inner div's transform is correctly rooted in the root
+  // frame's transform tree.
   // This asserts that we have the following tree structure:
   // ...
   //   Transform transform=translation=1.000000,2.000000,3.000000
   //     PreTranslation transform=translation=7.000000,7.000000,0.000000
   //       ScrollTranslation transform=translation=0.000000,0.000000,0.000000
   //         Transform transform=translation=4.000000,5.000000,6.000000
   auto* innerDocumentScrollTranslation = innerDivTransform->parent();
   EXPECT_EQ(TransformationMatrix().translate3d(0, 0, 0),
             innerDocumentScrollTranslation->matrix());
   auto* iframePreTranslation = innerDocumentScrollTranslation->parent();
@@ skipping 18 matching lines @@
                        "9px); width: 100px; height: 200px'></div>");
   FrameView* frameView = document().view();
   frameView->updateAllLifecyclePhases();
 
   // Assert that we have the following tree structure:
   // ...
   //   Transform transform=translation=1.000000,2.000000,3.000000
   //     PaintOffsetTranslation transform=translation=7.000000,7.000000,0.000000
   //       Transform transform=translation=4.000000,5.000000,6.000000
   //         PreTranslation transform=translation=42.000000,42.000000,0.000000
-  //           ScrollTranslation transform=translation=0.000000,0.000000,0.000000
-  //             PaintOffsetTranslation transform=translation=31.000000,31.000000,0.000000
+  //           ScrollTranslation transform=translation=0.000000,0.000000,0.00000
+  //             PaintOffsetTranslation transform=translation=31.00,31.00,0.00
   //               Transform transform=translation=7.000000,8.000000,9.000000
 
   LayoutObject* innerDivWithTransform =
       frameDocument.getElementById("transform")->layoutObject();
   auto* innerDivTransform =
       innerDivWithTransform->objectPaintProperties()->transform();
   EXPECT_EQ(TransformationMatrix().translate3d(7, 8, 9),
             innerDivTransform->matrix());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(92, 95, 100, 111), innerDivWithTransform,
                           frameView->layoutView());
@@ skipping 23 matching lines @@
   EXPECT_EQ(divWithTransformTransform,
             divWithTransform->objectPaintProperties()->transform());
   // http://crbug.com/638415
   if (!RuntimeEnabledFeatures::rootLayerScrollingEnabled()) {
     CHECK_EXACT_VISUAL_RECT(LayoutRect(1, 2, 800, 248), divWithTransform,
                             frameView->layoutView());
   }
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, TreeContextClipByNonStackingContext) {
-  // This test verifies the tree builder correctly computes and records the property tree context
-  // for a (pseudo) stacking context that is scrolled by a containing block that is not one of
-  // the painting ancestors.
+  // This test verifies the tree builder correctly computes and records the
+  // property tree context for a (pseudo) stacking context that is scrolled by a
+  // containing block that is not one of the painting ancestors.
   setBodyInnerHTML(
       "<style>body { margin: 0; }</style>"
       "<div id='scroller' style='overflow:scroll; width:400px; height:300px;'>"
       "  <div id='child' style='position:relative; width:100px; height: "
       "200px;'></div>"
       "  <div style='height:10000px;'></div>"
       "</div>");
   FrameView* frameView = document().view();
 
   LayoutObject* scroller =
@@ skipping 13 matching lines @@
       nullptr,
       childProperties->localBorderBoxProperties()->propertyTreeState.effect());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(0, 0, 400, 300), scroller,
                           frameView->layoutView());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(0, 0, 100, 200), child,
                           frameView->layoutView());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest,
        TreeContextUnclipFromParentStackingContext) {
-  // This test verifies the tree builder correctly computes and records the property tree context
-  // for a (pseudo) stacking context that has a scrolling painting ancestor that is not its
-  // containing block (thus should not be scrolled by it).
+  // This test verifies the tree builder correctly computes and records the
+  // property tree context for a (pseudo) stacking context that has a scrolling
+  // painting ancestor that is not its containing block (thus should not be
+  // scrolled by it).
 
   setBodyInnerHTML(
       "<style>body { margin: 0; }</style>"
       "<div id='scroller' style='overflow:scroll; opacity:0.5;'>"
       "  <div id='child' style='position:absolute; left:0; top:0; width: "
       "100px; height: 200px'></div>"
       "  <div style='height:10000px;'></div>"
       "</div>");
 
   LayoutObject& scroller =
@@ skipping 14 matching lines @@
       childProperties->localBorderBoxProperties()->propertyTreeState.effect());
   if (!RuntimeEnabledFeatures::rootLayerScrollingEnabled()) {
     CHECK_EXACT_VISUAL_RECT(LayoutRect(0, 0, 800, 10000), &scroller,
                             document().view()->layoutView());
   }
   CHECK_EXACT_VISUAL_RECT(LayoutRect(0, 0, 100, 200), &child,
                           document().view()->layoutView());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, TableCellLayoutLocation) {
-  // This test verifies that the border box space of a table cell is being correctly computed.
-  // Table cells have weird location adjustment in our layout/paint implementation.
+  // This test verifies that the border box space of a table cell is being
+  // correctly computed.  Table cells have weird location adjustment in our
+  // layout/paint implementation.
   setBodyInnerHTML(
       "<style>"
       "  body {"
       "    margin: 0;"
       "  }"
       "  table {"
       "    border-spacing: 0;"
       "    margin: 20px;"
       "    padding: 40px;"
       "    border: 10px solid black;"
@@ skipping 20 matching lines @@
 
   EXPECT_EQ(LayoutPoint(170, 170),
             targetProperties->localBorderBoxProperties()->paintOffset);
   EXPECT_EQ(framePreTranslation(), targetProperties->localBorderBoxProperties()
                                        ->propertyTreeState.transform());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(170, 170, 100, 100), &target,
                           document().view()->layoutView());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, CSSClipFixedPositionDescendant) {
-  // This test verifies that clip tree hierarchy being generated correctly for the hard case
-  // such that a fixed position element getting clipped by an absolute position CSS clip.
+  // This test verifies that clip tree hierarchy being generated correctly for
+  // the hard case such that a fixed position element getting clipped by an
+  // absolute position CSS clip.
   setBodyInnerHTML(
       "<style>"
       "  #clip {"
       "    position: absolute;"
       "    left: 123px;"
       "    top: 456px;"
       "    clip: rect(10px, 80px, 70px, 40px);"
       "    width: 100px;"
       "    height: 100px;"
       "  }"
@@ skipping 10 matching lines @@
   LayoutRect absoluteClipRect = localClipRect;
   absoluteClipRect.move(123, 456);
 
   LayoutObject& clip = *document().getElementById("clip")->layoutObject();
   const ObjectPaintProperties* clipProperties = clip.objectPaintProperties();
   EXPECT_EQ(frameContentClip(), clipProperties->cssClip()->parent());
   EXPECT_EQ(framePreTranslation(),
             clipProperties->cssClip()->localTransformSpace());
   EXPECT_EQ(FloatRoundedRect(FloatRect(absoluteClipRect)),
             clipProperties->cssClip()->clipRect());
-  CHECK_VISUAL_RECT(
-      absoluteClipRect, &clip, document().view()->layoutView(),
-      // TODO(crbug.com/599939): mapToVisualRectInAncestorSpace() doesn't apply css clip on the object itself.
-      LayoutUnit::max());
+  CHECK_VISUAL_RECT(absoluteClipRect, &clip, document().view()->layoutView(),
+                    // TODO(crbug.com/599939): mapToVisualRectInAncestorSpace()
+                    // doesn't apply css clip on the object itself.
+                    LayoutUnit::max());
 
   LayoutObject* fixed = document().getElementById("fixed")->layoutObject();
   const ObjectPaintProperties* fixedProperties = fixed->objectPaintProperties();
   EXPECT_EQ(
       clipProperties->cssClip(),
       fixedProperties->localBorderBoxProperties()->propertyTreeState.clip());
   EXPECT_EQ(framePreTranslation(), fixedProperties->localBorderBoxProperties()
                                        ->propertyTreeState.transform()
                                        ->parent());
   EXPECT_EQ(TransformationMatrix().translate(654, 321),
             fixedProperties->localBorderBoxProperties()
                 ->propertyTreeState.transform()
                 ->matrix());
   EXPECT_EQ(LayoutPoint(),
             fixedProperties->localBorderBoxProperties()->paintOffset);
-  CHECK_VISUAL_RECT(
-      LayoutRect(), fixed, document().view()->layoutView(),
-      // TODO(crbug.com/599939): CSS clip of fixed-position descendants is broken in mapToVisualRectInAncestorSpace().
-      LayoutUnit::max());
+  CHECK_VISUAL_RECT(LayoutRect(), fixed, document().view()->layoutView(),
+                    // TODO(crbug.com/599939): CSS clip of fixed-position
+                    // descendants is broken in
+                    // mapToVisualRectInAncestorSpace().
+                    LayoutUnit::max());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, CSSClipAbsPositionDescendant) {
-  // This test verifies that clip tree hierarchy being generated correctly for the hard case
-  // such that a fixed position element getting clipped by an absolute position CSS clip.
+  // This test verifies that clip tree hierarchy being generated correctly for
+  // the hard case such that a fixed position element getting clipped by an
+  // absolute position CSS clip.
   setBodyInnerHTML(
       "<style>"
       "  #clip {"
       "    position: absolute;"
       "    left: 123px;"
       "    top: 456px;"
       "    clip: rect(10px, 80px, 70px, 40px);"
       "    width: 100px;"
       "    height: 100px;"
       "  }"
       "  #abs {"
       "    position: absolute;"
       "    left: 654px;"
       "    top: 321px;"
       "    width: 10px;"
       "    heght: 20px"
       "  }"
       "</style>"
       "<div id='clip'><div id='absolute'></div></div>");
 
   LayoutRect localClipRect(40, 10, 40, 60);
   LayoutRect absoluteClipRect = localClipRect;
   absoluteClipRect.move(123, 456);
 
   LayoutObject* clip = document().getElementById("clip")->layoutObject();
   const ObjectPaintProperties* clipProperties = clip->objectPaintProperties();
   EXPECT_EQ(frameContentClip(), clipProperties->cssClip()->parent());
-  // No scroll translation because the document does not scroll (not enough content).
+  // No scroll translation because the document does not scroll (not enough
+  // content).
   EXPECT_TRUE(!frameScrollTranslation());
   EXPECT_EQ(framePreTranslation(),
             clipProperties->cssClip()->localTransformSpace());
   EXPECT_EQ(FloatRoundedRect(FloatRect(absoluteClipRect)),
             clipProperties->cssClip()->clipRect());
-  CHECK_VISUAL_RECT(
-      absoluteClipRect, clip, document().view()->layoutView(),
-      // TODO(crbug.com/599939): mapToVisualRectInAncestorSpace() doesn't apply css clip on the object itself.
-      LayoutUnit::max());
+  CHECK_VISUAL_RECT(absoluteClipRect, clip, document().view()->layoutView(),
+                    // TODO(crbug.com/599939): mapToVisualRectInAncestorSpace()
+                    // doesn't apply css clip on the object itself.
+                    LayoutUnit::max());
 
   LayoutObject* absolute =
       document().getElementById("absolute")->layoutObject();
   const ObjectPaintProperties* absPosProperties =
       absolute->objectPaintProperties();
   EXPECT_EQ(
       clipProperties->cssClip(),
       absPosProperties->localBorderBoxProperties()->propertyTreeState.clip());
   EXPECT_EQ(framePreTranslation(), absPosProperties->localBorderBoxProperties()
                                        ->propertyTreeState.transform());
   EXPECT_EQ(LayoutPoint(123, 456),
             absPosProperties->localBorderBoxProperties()->paintOffset);
-  CHECK_VISUAL_RECT(
-      LayoutRect(), absolute, document().view()->layoutView(),
-      // TODO(crbug.com/599939): CSS clip of fixed-position descendants is broken in mapToVisualRectInAncestorSpace().
-      LayoutUnit::max());
+  CHECK_VISUAL_RECT(LayoutRect(), absolute, document().view()->layoutView(),
+                    // TODO(crbug.com/599939): CSS clip of fixed-position
+                    // descendants is broken in
+                    // mapToVisualRectInAncestorSpace().
+                    LayoutUnit::max());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, CSSClipFixedPositionDescendantNonShared) {
   // This test is similar to CSSClipFixedPositionDescendant above, except that
-  // now we have a parent overflow clip that should be escaped by the fixed descendant.
+  // now we have a parent overflow clip that should be escaped by the fixed
+  // descendant.
   setBodyInnerHTML(
       "<style>"
       "  body {"
       "    margin: 0;"
       "  }"
       "  #overflow {"
       "    position: relative;"
       "    width: 50px;"
       "    height: 50px;"
       "    overflow: scroll;"
@@ skipping 15 matching lines @@
       "<div id='overflow'><div id='clip'><div id='fixed'></div></div></div>");
   LayoutRect localClipRect(40, 10, 40, 60);
   LayoutRect absoluteClipRect = localClipRect;
   absoluteClipRect.move(123, 456);
 
   LayoutObject& overflow =
       *document().getElementById("overflow")->layoutObject();
   const ObjectPaintProperties* overflowProperties =
       overflow.objectPaintProperties();
   EXPECT_EQ(frameContentClip(), overflowProperties->overflowClip()->parent());
-  // No scroll translation because the document does not scroll (not enough content).
+  // No scroll translation because the document does not scroll (not enough
+  // content).
   EXPECT_TRUE(!frameScrollTranslation());
   EXPECT_EQ(framePreTranslation(),
             overflowProperties->scrollTranslation()->parent());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(0, 0, 50, 50), &overflow,
                           document().view()->layoutView());
 
   LayoutObject* clip = document().getElementById("clip")->layoutObject();
   const ObjectPaintProperties* clipProperties = clip->objectPaintProperties();
   EXPECT_EQ(overflowProperties->overflowClip(),
             clipProperties->cssClip()->parent());
@@ skipping 16 matching lines @@
       fixedProperties->localBorderBoxProperties()->propertyTreeState.clip());
   EXPECT_EQ(framePreTranslation(), fixedProperties->localBorderBoxProperties()
                                        ->propertyTreeState.transform()
                                        ->parent());
   EXPECT_EQ(TransformationMatrix().translate(654, 321),
             fixedProperties->localBorderBoxProperties()
                 ->propertyTreeState.transform()
                 ->matrix());
   EXPECT_EQ(LayoutPoint(),
             fixedProperties->localBorderBoxProperties()->paintOffset);
-  CHECK_VISUAL_RECT(
-      LayoutRect(), fixed, document().view()->layoutView(),
-      // TODO(crbug.com/599939): CSS clip of fixed-position descendants is broken in geometry mapping.
-      LayoutUnit::max());
+  CHECK_VISUAL_RECT(LayoutRect(), fixed, document().view()->layoutView(),
+                    // TODO(crbug.com/599939): CSS clip of fixed-position
+                    // descendants is broken in geometry mapping.
+                    LayoutUnit::max());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, ColumnSpannerUnderRelativePositioned) {
   setBodyInnerHTML(
       "<div style='columns: 3; position: absolute; top: 44px; left: 55px;'>"
       "  <div style='position: relative; top: 100px; left: 100px'>"
       "    <div id='spanner' style='column-span: all; opacity: 0.5; width: "
       "100px; height: 100px;'></div>"
       "  </div>"
       "</div>");
@@ skipping 57 matching lines @@
   // The paint offset transform should be snapped from (0.3,0.3) to (0,0).
   EXPECT_EQ(TransformationMatrix().translate(0, 0),
             bProperties->transform()->parent()->matrix());
   // The residual subpixel adjustment should be (0.3,0.3) - (0,0) = (0.3,0.3).
   LayoutPoint subpixelAccumulation = LayoutPoint(FloatPoint(0.3, 0.3));
   EXPECT_EQ(subpixelAccumulation,
             bProperties->localBorderBoxProperties()->paintOffset);
   CHECK_EXACT_VISUAL_RECT(LayoutRect(FloatRect(0.3, 0.3, 40, 40)), b,
                           frameView->layoutView());
 
-  // c should be painted starting at subpixelAccumulation + (0.1,0.1) = (0.4,0.4).
+  // c should be painted starting at subpixelAccumulation + (0.1,0.1) =
+  // (0.4,0.4).
   LayoutObject* c = document().getElementById("c")->layoutObject();
   LayoutPoint cPaintOffset =
       subpixelAccumulation + LayoutPoint(FloatPoint(0.1, 0.1));
   const ObjectPaintProperties* cProperties = c->objectPaintProperties();
   EXPECT_EQ(cPaintOffset, cProperties->localBorderBoxProperties()->paintOffset);
-  // Visual rects via the non-paint properties system use enclosingIntRect before applying transforms,
-  // because they are computed bottom-up and therefore can't apply pixel snapping. Therefore apply a
-  // slop of 1px.
+  // Visual rects via the non-paint properties system use enclosingIntRect
+  // before applying transforms, because they are computed bottom-up and
+  // therefore can't apply pixel snapping. Therefore apply a slop of 1px.
   CHECK_VISUAL_RECT(LayoutRect(FloatRect(0.4, 0.4, 40, 40)), c,
                     frameView->layoutView(), 1);
 }
 
 TEST_P(PaintPropertyTreeBuilderTest,
        PaintOffsetWithPixelSnappingThroughTransform) {
   setBodyInnerHTML(
       "<style>"
       "  * { margin: 0; }"
       "  div { position: relative; }"
@@ skipping 16 matching lines @@
             bProperties->transform()->parent()->matrix());
   // The residual subpixel adjustment should be (0.7,0.7) - (1,1) = (-0.3,-0.3).
   LayoutPoint subpixelAccumulation =
       LayoutPoint(LayoutPoint(FloatPoint(0.7, 0.7)) - LayoutPoint(1, 1));
   EXPECT_EQ(subpixelAccumulation,
             bProperties->localBorderBoxProperties()->paintOffset);
   CHECK_EXACT_VISUAL_RECT(LayoutRect(LayoutUnit(0.7), LayoutUnit(0.7),
                                      LayoutUnit(40), LayoutUnit(40)),
                           b, frameView->layoutView());
 
-  // c should be painted starting at subpixelAccumulation + (0.7,0.7) = (0.4,0.4).
+  // c should be painted starting at subpixelAccumulation + (0.7,0.7) =
+  // (0.4,0.4).
   LayoutObject* c = document().getElementById("c")->layoutObject();
   LayoutPoint cPaintOffset =
       subpixelAccumulation + LayoutPoint(FloatPoint(0.7, 0.7));
   const ObjectPaintProperties* cProperties = c->objectPaintProperties();
   EXPECT_EQ(cPaintOffset, cProperties->localBorderBoxProperties()->paintOffset);
-  // Visual rects via the non-paint properties system use enclosingIntRect before applying transforms,
-  // because they are computed bottom-up and therefore can't apply pixel snapping. Therefore apply a
-  // slop of 1px.
+  // Visual rects via the non-paint properties system use enclosingIntRect
+  // before applying transforms, because they are computed bottom-up and
+  // therefore can't apply pixel snapping. Therefore apply a slop of 1px.
   CHECK_VISUAL_RECT(LayoutRect(LayoutUnit(0.7) + LayoutUnit(0.7),
                                LayoutUnit(0.7) + LayoutUnit(0.7),
                                LayoutUnit(40), LayoutUnit(40)),
                     c, frameView->layoutView(), 1);
 }
 
 TEST_P(PaintPropertyTreeBuilderTest,
        PaintOffsetWithPixelSnappingThroughMultipleTransforms) {
   setBodyInnerHTML(
       "<style>"
@@ skipping 25 matching lines @@
   EXPECT_EQ(subpixelAccumulation,
             bProperties->localBorderBoxProperties()->paintOffset);
   CHECK_EXACT_VISUAL_RECT(LayoutRect(LayoutUnit(5.7), LayoutUnit(7.7),
                                      LayoutUnit(40), LayoutUnit(40)),
                           b, frameView->layoutView());
 
   LayoutObject* c = document().getElementById("c")->layoutObject();
   const ObjectPaintProperties* cProperties = c->objectPaintProperties();
   EXPECT_EQ(TransformationMatrix().translate3d(11, 13, 0),
             cProperties->transform()->matrix());
-  // The paint offset should be (-0.3,-0.3) but the paint offset transform should still be at
-  // (0,0) because it should be snapped.
+  // The paint offset should be (-0.3,-0.3) but the paint offset transform
+  // should still be at (0,0) because it should be snapped.
   EXPECT_EQ(TransformationMatrix().translate(0, 0),
             cProperties->transform()->parent()->matrix());
   // The residual subpixel adjustment should still be (-0.3,-0.3).
   EXPECT_EQ(subpixelAccumulation,
             cProperties->localBorderBoxProperties()->paintOffset);
   CHECK_EXACT_VISUAL_RECT(LayoutRect(LayoutUnit(16.7), LayoutUnit(20.7),
                                      LayoutUnit(40), LayoutUnit(40)),
                           c, frameView->layoutView());
 
-  // d should be painted starting at subpixelAccumulation + (0.7,0.7) = (0.4,0.4).
+  // d should be painted starting at subpixelAccumulation + (0.7,0.7) =
+  // (0.4,0.4).
   LayoutObject* d = document().getElementById("d")->layoutObject();
   LayoutPoint dPaintOffset =
       subpixelAccumulation + LayoutPoint(FloatPoint(0.7, 0.7));
   const ObjectPaintProperties* dProperties = d->objectPaintProperties();
   EXPECT_EQ(dPaintOffset, dProperties->localBorderBoxProperties()->paintOffset);
-  // Visual rects via the non-paint properties system use enclosingIntRect before applying transforms,
-  // because they are computed bottom-up and therefore can't apply pixel snapping. Therefore apply a
-  // slop of 1px.
+  // Visual rects via the non-paint properties system use enclosingIntRect
+  // before applying transforms, because they are computed bottom-up and
+  // therefore can't apply pixel snapping. Therefore apply a slop of 1px.
   CHECK_VISUAL_RECT(LayoutRect(LayoutUnit(16.7) + LayoutUnit(0.7),
                                LayoutUnit(20.7) + LayoutUnit(0.7),
                                LayoutUnit(40), LayoutUnit(40)),
                     d, frameView->layoutView(), 1);
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, PaintOffsetWithPixelSnappingWithFixedPos) {
   setBodyInnerHTML(
       "<style>"
       "  * { margin: 0; }"
@@ skipping 34 matching lines @@
   CHECK_EXACT_VISUAL_RECT(LayoutRect(LayoutUnit(0.7), LayoutUnit(0),
                                      LayoutUnit(40), LayoutUnit(40)),
                           fixed, frameView->layoutView());
 
   // d should be painted starting at subpixelAccumulation + (0.7,0) = (0.4,0).
   LayoutObject* d = document().getElementById("d")->layoutObject();
   LayoutPoint dPaintOffset =
       subpixelAccumulation + LayoutPoint(FloatPoint(0.7, 0));
   const ObjectPaintProperties* dProperties = d->objectPaintProperties();
   EXPECT_EQ(dPaintOffset, dProperties->localBorderBoxProperties()->paintOffset);
-  // Visual rects via the non-paint properties system use enclosingIntRect before applying transforms,
-  // because they are computed bottom-up and therefore can't apply pixel snapping. Therefore apply a
-  // slop of 1px.
+  // Visual rects via the non-paint properties system use enclosingIntRect
+  // before applying transforms, because they are computed bottom-up and
+  // therefore can't apply pixel snapping. Therefore apply a slop of 1px.
   CHECK_VISUAL_RECT(LayoutRect(LayoutUnit(0.7) + LayoutUnit(0.7), LayoutUnit(),
                                LayoutUnit(40), LayoutUnit(40)),
                     d, frameView->layoutView(), 1);
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, SvgPixelSnappingShouldResetPaintOffset) {
   setBodyInnerHTML(
       "<svg id='svg' style='position: relative; left: 0.1px; transform: "
       "matrix(1, 0, 0, 1, 0, 0);'>"
       "    <rect id='rect' transform='translate(1, 1)'/>"
@@ skipping 11 matching lines @@
   EXPECT_EQ(nullptr,
             svgWithTransformProperties->svgLocalToBorderBoxTransform());
 
   LayoutObject& rectWithTransform =
       *document().getElementById("rect")->layoutObject();
   const ObjectPaintProperties* rectWithTransformProperties =
       rectWithTransform.objectPaintProperties();
   EXPECT_EQ(TransformationMatrix().translate(1, 1),
             rectWithTransformProperties->transform()->matrix());
 
-  // Ensure there is no PaintOffset transform between the rect and the svg's transform.
+  // Ensure there is no PaintOffset transform between the rect and the svg's
+  // transform.
   EXPECT_EQ(svgWithTransformProperties->transform(),
             rectWithTransformProperties->transform()->parent());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, NoRenderingContextByDefault) {
   setBodyInnerHTML("<div style='transform: translateZ(0)'></div>");
 
   const ObjectPaintProperties* properties =
       document().body()->firstChild()->layoutObject()->objectPaintProperties();
   ASSERT_TRUE(properties->transform());
@@ skipping 240 matching lines @@
   const TransformPaintPropertyNode* cTransformNode = cProperties->transform();
   EXPECT_EQ(TransformationMatrix().translate(77, 88), cTransformNode->matrix());
 
   CHECK_EXACT_VISUAL_RECT(LayoutRect(33, 44, 50, 60), a->layoutObject(),
                           frameView->layoutView());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(88, 110, 30, 40), b->layoutObject(),
                           frameView->layoutView());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(165, 198, 10, 20), c->layoutObject(),
                           frameView->layoutView());
 
-  // Change transform of b. B's transform node should be a new node with the new value,
-  // and a and c's transform nodes should be unchanged (with c's parent adjusted).
+  // Change transform of b. B's transform node should be a new node with the new
+  // value, and a and c's transform nodes should be unchanged (with c's parent
+  // adjusted).
   b->setAttribute(HTMLNames::styleAttr, "transform: translate(111px, 222px)");
   document().view()->updateAllLifecyclePhases();
 
   EXPECT_EQ(aProperties, a->layoutObject()->objectPaintProperties());
   EXPECT_EQ(aTransformNode, aProperties->transform());
 
   EXPECT_EQ(bProperties, b->layoutObject()->objectPaintProperties());
   bTransformNode = bProperties->transform();
   EXPECT_EQ(TransformationMatrix().translate(111, 222),
             bTransformNode->matrix());
   EXPECT_EQ(aTransformNode, bTransformNode->parent());
 
   EXPECT_EQ(cProperties, c->layoutObject()->objectPaintProperties());
   EXPECT_EQ(cTransformNode, cProperties->transform());
   EXPECT_EQ(bTransformNode, cTransformNode->parent());
 
   CHECK_EXACT_VISUAL_RECT(LayoutRect(33, 44, 50, 60), a->layoutObject(),
                           frameView->layoutView());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(144, 266, 50, 20), b->layoutObject(),
                           frameView->layoutView());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(221, 354, 10, 20), c->layoutObject(),
                           frameView->layoutView());
 
-  // Remove transform from b. B's transform node should be removed from the tree,
-  // and a and c's transform nodes should be unchanged (with c's parent adjusted).
+  // Remove transform from b. B's transform node should be removed from the
+  // tree, and a and c's transform nodes should be unchanged (with c's parent
+  // adjusted).
   b->setAttribute(HTMLNames::styleAttr, "");
   document().view()->updateAllLifecyclePhases();
 
   EXPECT_EQ(aProperties, a->layoutObject()->objectPaintProperties());
   EXPECT_EQ(aTransformNode, aProperties->transform());
 
   EXPECT_EQ(bProperties, b->layoutObject()->objectPaintProperties());
   EXPECT_EQ(nullptr, bProperties->transform());
 
   EXPECT_EQ(cProperties, c->layoutObject()->objectPaintProperties());
   EXPECT_EQ(cTransformNode, cProperties->transform());
   EXPECT_EQ(aTransformNode, cTransformNode->parent());
 
   CHECK_EXACT_VISUAL_RECT(LayoutRect(33, 44, 50, 60), a->layoutObject(),
                           frameView->layoutView());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(33, 44, 50, 20), b->layoutObject(),
                           frameView->layoutView());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(110, 132, 10, 20), c->layoutObject(),
                           frameView->layoutView());
 
   // Re-add transform to b. B's transform node should be inserted into the tree,
-  // and a and c's transform nodes should be unchanged (with c's parent adjusted).
+  // and a and c's transform nodes should be unchanged (with c's parent
+  // adjusted).
   b->setAttribute(HTMLNames::styleAttr, "transform: translate(4px, 5px)");
   document().view()->updateAllLifecyclePhases();
 
   EXPECT_EQ(aProperties, a->layoutObject()->objectPaintProperties());
   EXPECT_EQ(aTransformNode, aProperties->transform());
 
   EXPECT_EQ(bProperties, b->layoutObject()->objectPaintProperties());
   bTransformNode = bProperties->transform();
   EXPECT_EQ(TransformationMatrix().translate(4, 5), bTransformNode->matrix());
   EXPECT_EQ(aTransformNode, bTransformNode->parent());
 
   EXPECT_EQ(cProperties, c->layoutObject()->objectPaintProperties());
   EXPECT_EQ(cTransformNode, cProperties->transform());
   EXPECT_EQ(bTransformNode, cTransformNode->parent());
 
   CHECK_EXACT_VISUAL_RECT(LayoutRect(33, 44, 50, 60), a->layoutObject(),
                           frameView->layoutView());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(37, 49, 50, 20), b->layoutObject(),
                           frameView->layoutView());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(114, 137, 10, 20), c->layoutObject(),
                           frameView->layoutView());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, OverflowClipContentsTreeState) {
-  // This test verifies the tree builder correctly computes and records the property tree context
-  // for a (pseudo) stacking context that is scrolled by a containing block that is not one of
-  // the painting ancestors.
+  // This test verifies the tree builder correctly computes and records the
+  // property tree context for a (pseudo) stacking context that is scrolled by a
+  // containing block that is not one of the painting ancestors.
   setBodyInnerHTML(
       "<style>body { margin: 20px 30px; }</style>"
       "<div id='clipper' style='overflow:hidden; width:400px; height:300px;'>"
       "  <div id='child' style='position:relative; width:500px; height: "
       "600px;'></div>"
       "</div>");
 
   LayoutBoxModelObject* clipper = toLayoutBoxModelObject(
       document().getElementById("clipper")->layoutObject());
   const ObjectPaintProperties* clipProperties =
       clipper->objectPaintProperties();
   LayoutObject* child = document().getElementById("child")->layoutObject();
   const ObjectPaintProperties* childProperties = child->objectPaintProperties();
 
-  // No scroll translation because the document does not scroll (not enough content).
+  // No scroll translation because the document does not scroll (not enough
+  // content).
   EXPECT_TRUE(!frameScrollTranslation());
   EXPECT_EQ(framePreTranslation(), clipProperties->localBorderBoxProperties()
                                        ->propertyTreeState.transform());
   EXPECT_EQ(
       frameContentClip(),
       clipProperties->localBorderBoxProperties()->propertyTreeState.clip());
 
   auto contentsProperties = clipProperties->contentsProperties();
   EXPECT_EQ(LayoutPoint(30, 20), contentsProperties.paintOffset);
   EXPECT_EQ(framePreTranslation(),
@@ skipping 21 matching lines @@
       "500px;'></div>"
       "</div>");
 
   LayoutBoxModelObject* clipper = toLayoutBoxModelObject(
       document().getElementById("clipper")->layoutObject());
   const ObjectPaintProperties* clipProperties =
       clipper->objectPaintProperties();
   LayoutObject* child = document().getElementById("child")->layoutObject();
   const ObjectPaintProperties* childProperties = child->objectPaintProperties();
 
-  // No scroll translation because the document does not scroll (not enough content).
+  // No scroll translation because the document does not scroll (not enough
+  // content).
   EXPECT_TRUE(!frameScrollTranslation());
   EXPECT_EQ(framePreTranslation(), clipProperties->localBorderBoxProperties()
                                        ->propertyTreeState.transform());
   EXPECT_EQ(
       frameContentClip(),
       clipProperties->localBorderBoxProperties()->propertyTreeState.clip());
 
   auto contentsProperties = clipProperties->contentsProperties();
   EXPECT_EQ(LayoutPoint(30, 20), contentsProperties.paintOffset);
   EXPECT_EQ(framePreTranslation(),
             contentsProperties.propertyTreeState.transform());
   EXPECT_EQ(clipProperties->overflowClip(),
             contentsProperties.propertyTreeState.clip());
 
   EXPECT_EQ(framePreTranslation(), childProperties->localBorderBoxProperties()
                                        ->propertyTreeState.transform());
   EXPECT_EQ(
       clipProperties->overflowClip(),
       childProperties->localBorderBoxProperties()->propertyTreeState.clip());
 
   EXPECT_NE(
       nullptr,
       childProperties->localBorderBoxProperties()->propertyTreeState.effect());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(0, 0, 400, 500), child, clipper);
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, OverflowScrollContentsTreeState) {
-  // This test verifies the tree builder correctly computes and records the property tree context
-  // for a (pseudo) stacking context that is scrolled by a containing block that is not one of
-  // the painting ancestors.
+  // This test verifies the tree builder correctly computes and records the
+  // property tree context for a (pseudo) stacking context that is scrolled by a
+  // containing block that is not one of the painting ancestors.
   setBodyInnerHTML(
       "<style>body { margin: 20px 30px; }</style>"
       "<div id='clipper' style='overflow:scroll; width:400px; height:300px;'>"
       "  <div id='child' style='position:relative; width:500px; height: "
       "600px;'></div>"
       "  <div style='width: 200px; height: 10000px'></div>"
       "</div>"
       "<div id='forceScroll' style='height: 4000px;'></div>");
 
   Element* clipperElement = document().getElementById("clipper");
@@ skipping 57 matching lines @@
 
   LayoutObject& roundedBox =
       *document().getElementById("roundedBox")->layoutObject();
   const ObjectPaintProperties* roundedBoxProperties =
       roundedBox.objectPaintProperties();
   EXPECT_EQ(
       FloatRoundedRect(FloatRect(50, 50, 200, 200), FloatSize(50, 50),
                        FloatSize(50, 50), FloatSize(50, 50), FloatSize(50, 50)),
       roundedBoxProperties->innerBorderRadiusClip()->clipRect());
 
-  // Unlike the inner border radius clip, the overflow clip is inset by the scrollbars (13px).
+  // Unlike the inner border radius clip, the overflow clip is inset by the
+  // scrollbars (13px).
   EXPECT_EQ(FloatRoundedRect(50, 50, 187, 187),
             roundedBoxProperties->overflowClip()->clipRect());
   EXPECT_EQ(frameContentClip(),
             roundedBoxProperties->innerBorderRadiusClip()->parent());
   EXPECT_EQ(roundedBoxProperties->innerBorderRadiusClip(),
             roundedBoxProperties->overflowClip()->parent());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, CssClipContentsTreeState) {
-  // This test verifies the tree builder correctly computes and records the property tree context
-  // for a (pseudo) stacking context that is scrolled by a containing block that is not one of
-  // the painting ancestors.
+  // This test verifies the tree builder correctly computes and records the
+  // property tree context for a (pseudo) stacking context that is scrolled by a
+  // containing block that is not one of the painting ancestors.
   setBodyInnerHTML(
       "<style>body { margin: 20px 30px; }</style>"
       "<div id='clipper' style='position: absolute; clip: rect(10px, 80px, "
       "70px, 40px); width:300px; height:200px;'>"
       "  <div id='child' style='position:relative; width:400px; height: "
       "500px;'></div>"
       "</div>");
 
   LayoutBoxModelObject* clipper = toLayoutBoxModelObject(
       document().getElementById("clipper")->layoutObject());
   const ObjectPaintProperties* clipProperties =
       clipper->objectPaintProperties();
   LayoutObject* child = document().getElementById("child")->layoutObject();
 
-  // No scroll translation because the document does not scroll (not enough content).
+  // No scroll translation because the document does not scroll (not enough
+  // content).
   EXPECT_TRUE(!frameScrollTranslation());
   EXPECT_EQ(framePreTranslation(), clipProperties->localBorderBoxProperties()
                                        ->propertyTreeState.transform());
   // CSS clip on an element causes it to clip itself, not just descendants.
   EXPECT_EQ(
       clipProperties->cssClip(),
       clipProperties->localBorderBoxProperties()->propertyTreeState.clip());
 
   auto contentsProperties = clipProperties->contentsProperties();
   EXPECT_EQ(LayoutPoint(30, 20), contentsProperties.paintOffset);
@@ skipping 58 matching lines @@
       "  <div class='forceScroll'></div>"
       "</div>");
 
   Element* overflowHidden = document().getElementById("overflowHidden");
   overflowHidden->setScrollTop(37);
 
   document().view()->updateAllLifecyclePhases();
 
   const ObjectPaintProperties* overflowHiddenScrollProperties =
       overflowHidden->layoutObject()->objectPaintProperties();
-  // Because the frameView is does not scroll, overflowHidden's scroll should be under the root.
+  // Because the frameView is does not scroll, overflowHidden's scroll should be
+  // under the root.
   EXPECT_TRUE(overflowHiddenScrollProperties->scroll()->parent()->isRoot());
   EXPECT_EQ(TransformationMatrix().translate(0, -37),
             overflowHiddenScrollProperties->scroll()
                 ->scrollOffsetTranslation()
                 ->matrix());
   // This should match the overflow's dimensions.
   EXPECT_EQ(IntSize(5, 3), overflowHiddenScrollProperties->scroll()->clip());
-  // The scrolling content's bounds should include both the overflow's dimensions (5x3) and the
-  // 0x79 "forceScroll" object.
+  // The scrolling content's bounds should include both the overflow's
+  // dimensions (5x3) and the 0x79 "forceScroll" object.
   EXPECT_EQ(IntSize(5, 79), overflowHiddenScrollProperties->scroll()->bounds());
-  // Although overflow: hidden is programmatically scrollable, it is not user scrollable.
+  // Although overflow: hidden is programmatically scrollable, it is not user
+  // scrollable.
   EXPECT_FALSE(
       overflowHiddenScrollProperties->scroll()->userScrollableHorizontal());
   EXPECT_FALSE(
       overflowHiddenScrollProperties->scroll()->userScrollableVertical());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, NestedScrollProperties) {
   setBodyInnerHTML(
       "<style>"
       "  * {"
@@ skipping 22 matching lines @@
 
   Element* overflowA = document().getElementById("overflowA");
   overflowA->setScrollTop(37);
   Element* overflowB = document().getElementById("overflowB");
   overflowB->setScrollTop(41);
 
   document().view()->updateAllLifecyclePhases();
 
   const ObjectPaintProperties* overflowAScrollProperties =
       overflowA->layoutObject()->objectPaintProperties();
-  // Because the frameView is does not scroll, overflowA's scroll should be under the root.
+  // Because the frameView is does not scroll, overflowA's scroll should be
+  // under the root.
   EXPECT_TRUE(overflowAScrollProperties->scroll()->parent()->isRoot());
   EXPECT_EQ(
       TransformationMatrix().translate(0, -37),
       overflowAScrollProperties->scroll()->scrollOffsetTranslation()->matrix());
   EXPECT_EQ(IntSize(5, 3), overflowAScrollProperties->scroll()->clip());
-  // 107 is the forceScroll element plus the height of the overflow scroll child (overflowB).
+  // 107 is the forceScroll element plus the height of the overflow scroll child
+  // (overflowB).
   EXPECT_EQ(IntSize(9, 107), overflowAScrollProperties->scroll()->bounds());
   EXPECT_TRUE(overflowAScrollProperties->scroll()->userScrollableHorizontal());
   EXPECT_TRUE(overflowAScrollProperties->scroll()->userScrollableVertical());
 
   const ObjectPaintProperties* overflowBScrollProperties =
       overflowB->layoutObject()->objectPaintProperties();
-  // The overflow child's scroll node should be a child of the parent's (overflowA) scroll node.
+  // The overflow child's scroll node should be a child of the parent's
+  // (overflowA) scroll node.
   EXPECT_EQ(overflowAScrollProperties->scroll(),
             overflowBScrollProperties->scroll()->parent());
   EXPECT_EQ(
       TransformationMatrix().translate(0, -41),
       overflowBScrollProperties->scroll()->scrollOffsetTranslation()->matrix());
   EXPECT_EQ(IntSize(9, 7), overflowBScrollProperties->scroll()->clip());
   EXPECT_EQ(IntSize(9, 100), overflowBScrollProperties->scroll()->bounds());
   EXPECT_TRUE(overflowBScrollProperties->scroll()->userScrollableHorizontal());
   EXPECT_TRUE(overflowBScrollProperties->scroll()->userScrollableVertical());
 }
@@ skipping 52 matching lines @@
   // The frame should scroll due to the "forceScroll" element.
   EXPECT_NE(nullptr, frameScroll());
 
   const ObjectPaintProperties* overflowScrollProperties =
       overflow->layoutObject()->objectPaintProperties();
   EXPECT_EQ(frameScroll(), overflowScrollProperties->scroll()->parent());
   EXPECT_EQ(
       TransformationMatrix().translate(0, -37),
       overflowScrollProperties->scroll()->scrollOffsetTranslation()->matrix());
   EXPECT_EQ(IntSize(5, 3), overflowScrollProperties->scroll()->clip());
-  // The height should be 4000px because the (dom-order) overflow children are positioned and do not
-  // contribute to the height. Only the 4000px "forceScroll" height is present.
+  // The height should be 4000px because the (dom-order) overflow children are
+  // positioned and do not contribute to the height. Only the 4000px
+  // "forceScroll" height is present.
   EXPECT_EQ(IntSize(5, 4000), overflowScrollProperties->scroll()->bounds());
 
   const ObjectPaintProperties* absposOverflowScrollProperties =
       absposOverflow->layoutObject()->objectPaintProperties();
-  // The absolute position overflow scroll node is parented under the frame, not the dom-order parent.
+  // The absolute position overflow scroll node is parented under the frame, not
+  // the dom-order parent.
   EXPECT_EQ(frameScroll(), absposOverflowScrollProperties->scroll()->parent());
   EXPECT_EQ(TransformationMatrix().translate(0, -41),
             absposOverflowScrollProperties->scroll()
                 ->scrollOffsetTranslation()
                 ->matrix());
   EXPECT_EQ(IntSize(9, 7), absposOverflowScrollProperties->scroll()->clip());
   EXPECT_EQ(IntSize(9, 4000),
             absposOverflowScrollProperties->scroll()->bounds());
 
   const ObjectPaintProperties* fixedOverflowScrollProperties =
       fixedOverflow->layoutObject()->objectPaintProperties();
-  // The fixed position overflow scroll node is parented under the root, not the dom-order parent or frame's scroll.
+  // The fixed position overflow scroll node is parented under the root, not the
+  // dom-order parent or frame's scroll.
   EXPECT_TRUE(fixedOverflowScrollProperties->scroll()->parent()->isRoot());
   EXPECT_EQ(TransformationMatrix().translate(0, -43),
             fixedOverflowScrollProperties->scroll()
                 ->scrollOffsetTranslation()
                 ->matrix());
   EXPECT_EQ(IntSize(13, 11), fixedOverflowScrollProperties->scroll()->clip());
   EXPECT_EQ(IntSize(13, 4000),
             fixedOverflowScrollProperties->scroll()->bounds());
 }
 
@@ skipping 32 matching lines @@
 
   Element* overflowA = document().getElementById("overflowA");
   overflowA->setScrollTop(37);
   Element* overflowB = document().getElementById("overflowB");
   overflowB->setScrollTop(41);
 
   document().view()->updateAllLifecyclePhases();
 
   const ObjectPaintProperties* overflowAScrollProperties =
       overflowA->layoutObject()->objectPaintProperties();
-  // Because the frameView is does not scroll, overflowA's scroll should be under the root.
+  // Because the frameView is does not scroll, overflowA's scroll should be
+  // under the root.
   EXPECT_TRUE(overflowAScrollProperties->scroll()->parent()->isRoot());
   EXPECT_EQ(
       TransformationMatrix().translate(0, -37),
       overflowAScrollProperties->scroll()->scrollOffsetTranslation()->matrix());
   EXPECT_EQ(IntSize(20, 20), overflowAScrollProperties->scroll()->clip());
-  // 100 is the forceScroll element's height because the overflow child does not contribute to the height.
+  // 100 is the forceScroll element's height because the overflow child does not
+  // contribute to the height.
   EXPECT_EQ(IntSize(20, 100), overflowAScrollProperties->scroll()->bounds());
   EXPECT_TRUE(overflowAScrollProperties->scroll()->userScrollableHorizontal());
   EXPECT_TRUE(overflowAScrollProperties->scroll()->userScrollableVertical());
 
   const ObjectPaintProperties* overflowBScrollProperties =
       overflowB->layoutObject()->objectPaintProperties();
-  // The overflow child's scroll node should be a child of the parent's (overflowA) scroll node.
+  // The overflow child's scroll node should be a child of the parent's
+  // (overflowA) scroll node.
   EXPECT_EQ(overflowAScrollProperties->scroll(),
             overflowBScrollProperties->scroll()->parent());
   EXPECT_EQ(
       TransformationMatrix().translate(0, -41),
       overflowBScrollProperties->scroll()->scrollOffsetTranslation()->matrix());
   EXPECT_EQ(IntSize(5, 3), overflowBScrollProperties->scroll()->clip());
   EXPECT_EQ(IntSize(5, 100), overflowBScrollProperties->scroll()->bounds());
   EXPECT_TRUE(overflowBScrollProperties->scroll()->userScrollableHorizontal());
   EXPECT_TRUE(overflowBScrollProperties->scroll()->userScrollableVertical());
 }
@@ skipping 221 matching lines @@
   EXPECT_FALSE(
       overflowB->layoutObject()
           ->objectPaintProperties()
           ->scroll()
           ->parent()
           ->hasMainThreadScrollingReasons(
               MainThreadScrollingReason::kHasBackgroundAttachmentFixedObjects));
 }
 
 }  // namespace blink