Rewrap comments to 80 columns in Source/platform/graphics/paint/.

third_party/WebKit/Source/platform/graphics/paint/PaintControllerTest.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 "platform/graphics/paint/PaintController.h"
 
 #include "platform/RuntimeEnabledFeatures.h"
 #include "platform/graphics/GraphicsContext.h"
 #include "platform/graphics/paint/ClipPathDisplayItem.h"
 #include "platform/graphics/paint/ClipPathRecorder.h"
@@ skipping 105 matching lines @@
   drawRect(context, client, drawingType, bound);
 }
 
 enum TestConfigurations {
   SPv1,
   SPv2,
   UnderInvalidationCheckingSPv1,
   UnderInvalidationCheckingSPv2,
 };
 
-// Tests using this class will be tested with under-invalidation-checking enabled and disabled.
+// Tests using this class will be tested with under-invalidation-checking
+// enabled and disabled.
 class PaintControllerTest
     : public PaintControllerTestBase,
       public testing::WithParamInterface<TestConfigurations> {
  public:
   PaintControllerTest()
       : m_rootPaintPropertyClient("root"),
         m_rootPaintChunkId(m_rootPaintPropertyClient,
                            DisplayItem::kUninitializedType) {}
 
  protected:
@@ skipping 246 matching lines @@
                       TestDisplayItem(first, foregroundDrawingType),
                       TestDisplayItem(unaffected, backgroundDrawingType),
                       TestDisplayItem(unaffected, foregroundDrawingType));
 
   if (RuntimeEnabledFeatures::slimmingPaintV2Enabled()) {
     EXPECT_EQ(1u, getPaintController().paintChunks().size());
     EXPECT_THAT(getPaintController().paintChunks()[0].rasterInvalidationRects,
                 UnorderedElementsAre(
                     FloatRect(100, 100, 100, 100),    // Old bounds of |first|.
                     FloatRect(100, 100, 100, 100)));  // New bounds of |first|.
-                                                      // No need to invalidate raster of |second| because the client (|first|) which swapped order
-    // with it has been invalidated.
+    // No need to invalidate raster of |second|, because the client (|first|)
+    // which swapped order with it has been invalidated.
   }
 }
 
 TEST_P(PaintControllerTest, UpdateNewItemInMiddle) {
   FakeDisplayItemClient first("first", LayoutRect(100, 100, 100, 100));
   FakeDisplayItemClient second("second", LayoutRect(100, 100, 50, 200));
   FakeDisplayItemClient third("third", LayoutRect(125, 100, 200, 50));
   GraphicsContext context(getPaintController());
   if (RuntimeEnabledFeatures::slimmingPaintV2Enabled())
     getPaintController().updateCurrentPaintChunkProperties(
@@ skipping 460 matching lines @@
                       TestDisplayItem(container1, foregroundDrawingType),
                       TestDisplayItem(container2, backgroundDrawingType),
                       TestDisplayItem(content2, backgroundDrawingType),
                       TestDisplayItem(content2, foregroundDrawingType),
                       TestDisplayItem(container2, foregroundDrawingType));
 
   if (RuntimeEnabledFeatures::slimmingPaintV2Enabled())
     getPaintController().updateCurrentPaintChunkProperties(
         &m_rootPaintChunkId, PaintChunkProperties());
 
-  // Simulate the situation when container1 e.g. gets a z-index that is now greater than container2,
+  // Simulate the situation when |container1| gets a z-index that is greater
+  // than that of |container2|.
   drawRect(context, container2, backgroundDrawingType,
            FloatRect(100, 200, 100, 100));
   drawRect(context, content2, backgroundDrawingType,
            FloatRect(100, 200, 50, 200));
   drawRect(context, content2, foregroundDrawingType,
            FloatRect(100, 200, 50, 200));
   drawRect(context, container2, foregroundDrawingType,
            FloatRect(100, 200, 100, 100));
   drawRect(context, container1, backgroundDrawingType,
            FloatRect(100, 100, 100, 100));
@@ skipping 13 matching lines @@
                       TestDisplayItem(container1, backgroundDrawingType),
                       TestDisplayItem(content1, backgroundDrawingType),
                       TestDisplayItem(content1, foregroundDrawingType),
                       TestDisplayItem(container1, foregroundDrawingType));
 
   if (RuntimeEnabledFeatures::slimmingPaintV2Enabled()) {
     EXPECT_EQ(1u, getPaintController().paintChunks().size());
     EXPECT_THAT(
         getPaintController().paintChunks()[0].rasterInvalidationRects,
         UnorderedElementsAre(
-            FloatRect(
-                100, 200, 100,
-                100),  // Bounds of |container2| which was moved behind |container1|.
-            FloatRect(
-                100, 200, 50,
-                200)));  // Bounds of |content2| which was moved along with |container2|.
+            FloatRect(100, 200, 100, 100),   // Bounds of |container2| which was
+                                             // moved behind |container1|.
+            FloatRect(100, 200, 50, 200)));  // Bounds of |content2| which was
+                                             // moved along with |container2|.
   }
 }
 
 TEST_P(PaintControllerTest, UpdateSwapOrderWithChildrenAndInvalidation) {
   FakeDisplayItemClient container1("container1",
                                    LayoutRect(100, 100, 100, 100));
   FakeDisplayItemClient content1("content1", LayoutRect(100, 100, 50, 200));
   FakeDisplayItemClient container2("container2",
                                    LayoutRect(100, 200, 100, 100));
   FakeDisplayItemClient content2("content2", LayoutRect(100, 200, 50, 200));
@@ skipping 27 matching lines @@
                       TestDisplayItem(container1, foregroundDrawingType),
                       TestDisplayItem(container2, backgroundDrawingType),
                       TestDisplayItem(content2, backgroundDrawingType),
                       TestDisplayItem(content2, foregroundDrawingType),
                       TestDisplayItem(container2, foregroundDrawingType));
 
   if (RuntimeEnabledFeatures::slimmingPaintV2Enabled())
     getPaintController().updateCurrentPaintChunkProperties(
         &m_rootPaintChunkId, PaintChunkProperties());
 
-  // Simulate the situation when container1 e.g. gets a z-index that is now greater than container2.
-  // and container1 is invalidated.
+  // Simulate the situation when |container1| gets a z-index that is greater
+  // than that of |container2|, and |container1| is invalidated.
   container1.setDisplayItemsUncached();
   drawRect(context, container2, backgroundDrawingType,
            FloatRect(100, 200, 100, 100));
   drawRect(context, content2, backgroundDrawingType,
            FloatRect(100, 200, 50, 200));
   drawRect(context, content2, foregroundDrawingType,
            FloatRect(100, 200, 50, 200));
   drawRect(context, container2, foregroundDrawingType,
            FloatRect(100, 200, 100, 100));
   drawRect(context, container1, backgroundDrawingType,
@@ skipping 14 matching lines @@
                       TestDisplayItem(container1, backgroundDrawingType),
                       TestDisplayItem(content1, backgroundDrawingType),
                       TestDisplayItem(content1, foregroundDrawingType),
                       TestDisplayItem(container1, foregroundDrawingType));
 
   if (RuntimeEnabledFeatures::slimmingPaintV2Enabled()) {
     EXPECT_EQ(1u, getPaintController().paintChunks().size());
     EXPECT_THAT(
         getPaintController().paintChunks()[0].rasterInvalidationRects,
         UnorderedElementsAre(
-            FloatRect(100, 100, 100, 100),  // Old bounds of |container1|.
-            FloatRect(100, 100, 100, 100),  // New bounds of |container1|.
-            FloatRect(
-                100, 200, 100,
-                100),  // Bounds of |container2| which was moved behind |container1|.
-            FloatRect(
-                100, 200, 50,
-                200)));  // Bounds of |content2| which was moved along with |container2|.
+            FloatRect(100, 100, 100, 100),   // Old bounds of |container1|.
+            FloatRect(100, 100, 100, 100),   // New bounds of |container1|.
+            FloatRect(100, 200, 100, 100),   // Bounds of |container2| which was
+                                             // moved behind |container1|.
+            FloatRect(100, 200, 50, 200)));  // Bounds of |content2| which was
+                                             // moved along with |container2|.
   }
 }
 
 TEST_P(PaintControllerTest, CachedSubsequenceSwapOrder) {
   FakeDisplayItemClient container1("container1",
                                    LayoutRect(100, 100, 100, 100));
   FakeDisplayItemClient content1("content1", LayoutRect(100, 100, 50, 200));
   FakeDisplayItemClient container2("container2",
                                    LayoutRect(100, 200, 100, 100));
   FakeDisplayItemClient content2("content2", LayoutRect(100, 200, 50, 200));
@@ skipping 61 matching lines @@
     EXPECT_EQ(PaintChunk::Id(container1, backgroundDrawingType),
               getPaintController().paintChunks()[0].id);
     EXPECT_EQ(PaintChunk::Id(container2, backgroundDrawingType),
               getPaintController().paintChunks()[1].id);
     EXPECT_THAT(getPaintController().paintChunks()[0].rasterInvalidationRects,
                 UnorderedElementsAre(FloatRect(LayoutRect::infiniteIntRect())));
     EXPECT_THAT(getPaintController().paintChunks()[1].rasterInvalidationRects,
                 UnorderedElementsAre(FloatRect(LayoutRect::infiniteIntRect())));
   }
 
-  // Simulate the situation when container1 e.g. gets a z-index that is now greater than container2.
+  // Simulate the situation when |container1| gets a z-index that is greater than
+  // that of |container2|.
   if (RuntimeEnabledFeatures::paintUnderInvalidationCheckingEnabled()) {
-    // When under-invalidation-checking is enabled, useCachedSubsequenceIfPossible is forced off,
-    // and the client is expected to create the same painting as in the previous paint.
+    // When under-invalidation-checking is enabled,
+    // useCachedSubsequenceIfPossible is forced off, and the client is expected
+    // to create the same painting as in the previous paint.
     EXPECT_FALSE(SubsequenceRecorder::useCachedSubsequenceIfPossible(
         context, container2));
     {
       if (RuntimeEnabledFeatures::slimmingPaintV2Enabled()) {
         PaintChunk::Id id(container2, backgroundDrawingType);
         getPaintController().updateCurrentPaintChunkProperties(
             &id, container2Properties);
       }
       SubsequenceRecorder r(context, container2);
       drawRect(context, container2, backgroundDrawingType,
@@ skipping 348 matching lines @@
     drawRect(context, content2, foregroundDrawingType,
              FloatRect(100, 200, 50, 200));
   }
   // Repaint container1 with foreground only.
   {
     EXPECT_FALSE(SubsequenceRecorder::useCachedSubsequenceIfPossible(
         context, container1));
     SubsequenceRecorder r(context, container1);
     // Use cached subsequence of content1.
     if (RuntimeEnabledFeatures::paintUnderInvalidationCheckingEnabled()) {
-      // When under-invalidation-checking is enabled, useCachedSubsequenceIfPossible is forced off,
-      // and the client is expected to create the same painting as in the previous paint.
+      // When under-invalidation-checking is enabled,
+      // useCachedSubsequenceIfPossible is forced off, and the client is
+      // expected to create the same painting as in the previous paint.
       EXPECT_FALSE(SubsequenceRecorder::useCachedSubsequenceIfPossible(
           context, content1));
       if (RuntimeEnabledFeatures::slimmingPaintV2Enabled()) {
         PaintChunk::Id id(content1, backgroundDrawingType);
         getPaintController().updateCurrentPaintChunkProperties(
             &id, content1Properties);
       }
       SubsequenceRecorder r(context, content1);
       drawRect(context, content1, backgroundDrawingType,
                FloatRect(100, 100, 50, 200));
@@ skipping 454 matching lines @@
       SubsequenceRecorder::useCachedSubsequenceIfPossible(context, container));
   getPaintController().commitNewDisplayItems(LayoutSize());
   EXPECT_FALSE(
       getPaintController().paintArtifact().isSuitableForGpuRasterization());
 
 #if CHECK_DISPLAY_ITEM_CLIENT_ALIVENESS
   DisplayItemClient::endShouldKeepAliveAllClients();
 #endif
 }
 
-// Temporarily disabled (pref regressions due to GPU veto stickiness: http://crbug.com/603969).
+// Temporarily disabled (pref regressions due to GPU veto stickiness:
+// http://crbug.com/603969).
 TEST_F(PaintControllerTestBase,
        DISABLED_IsNotSuitableForGpuRasterizationConcaveClipPath) {
   Path path;
   path.addLineTo(FloatPoint(50, 50));
   path.addLineTo(FloatPoint(100, 0));
   path.addLineTo(FloatPoint(50, 100));
   path.closeSubpath();
 
   FakeDisplayItemClient client("test client", LayoutRect(0, 0, 200, 100));
   GraphicsContext context(getPaintController());
@@ skipping 73 matching lines @@
     {
       SubsequenceRecorder r(context, container);
       drawRect(context, container, backgroundDrawingType,
                FloatRect(100, 100, 100, 100));
     }
     getPaintController().commitNewDisplayItems();
 
     EXPECT_FALSE(SubsequenceRecorder::useCachedSubsequenceIfPossible(
         context, container));
     {
-      // Generate some no-op pairs which should not affect under-invalidation checking.
+      // Generate some no-op pairs which should not affect under-invalidation
+      // checking.
       ClipRecorder r1(context, container, clipType, IntRect(1, 1, 9, 9));
       ClipRecorder r2(context, container, clipType, IntRect(1, 1, 2, 2));
       ClipRecorder r3(context, container, clipType, IntRect(1, 1, 3, 3));
       ClipPathRecorder r4(context, container, Path());
     }
     {
       EXPECT_FALSE(SubsequenceRecorder::useCachedSubsequenceIfPossible(
           context, container));
       SubsequenceRecorder r(context, container);
       drawRect(context, container, backgroundDrawingType,
@@ skipping 166 matching lines @@
 
 TEST_F(PaintControllerUnderInvalidationTest, ChangeDrawing) {
   EXPECT_DEATH(testChangeDrawing(), "under-invalidation: display item changed");
 }
 
 TEST_F(PaintControllerUnderInvalidationTest, MoreDrawing) {
   EXPECT_DEATH(testMoreDrawing(), "");
 }
 
 TEST_F(PaintControllerUnderInvalidationTest, LessDrawing) {
-  // We don't detect under-invalidation in this case, and PaintController can also handle the case gracefully.
-  // However, less-drawing at a time often means more-drawing at another time so eventually we'll detect
-  // such under-invalidations.
+  // We don't detect under-invalidation in this case, and PaintController can
+  // also handle the case gracefully. However, less drawing at one time often
+  // means more-drawing at another time, so eventually we'll detect such
+  // under-invalidations.
   testLessDrawing();
 }
 
 TEST_F(PaintControllerUnderInvalidationTest, NoopPairsInSubsequence) {
   // This should not die.
   testNoopPairsInSubsequence();
 }
 
 TEST_F(PaintControllerUnderInvalidationTest, ChangeDrawingInSubsequence) {
   EXPECT_DEATH(testChangeDrawingInSubsequence(),
@@ skipping 13 matching lines @@
                "display item changed");
 }
 
 TEST_F(PaintControllerUnderInvalidationTest, ChangeNonCacheableInSubsequence) {
   EXPECT_DEATH(testChangeNonCacheableInSubsequence(),
                "\"\\(In cached subsequence of container\\)\" "
                "under-invalidation: display item changed");
 }
 
 TEST_F(PaintControllerUnderInvalidationTest, InvalidationInSubsequence) {
-  // We allow invalidated display item clients as long as they would produce the same display items.
-  // The cases of changed display items are tested by other test cases.
+  // We allow invalidated display item clients as long as they would produce the
+  // same display items. The cases of changed display items are tested by other
+  // test cases.
   testInvalidationInSubsequence();
 }
 
 TEST_F(PaintControllerUnderInvalidationTest,
        FoldCompositingDrawingInSubsequence) {
   testFoldCompositingDrawingInSubsequence();
 }
 
 #endif  // defined(GTEST_HAS_DEATH_TEST) && !OS(ANDROID)
 
 }  // namespace blink