reflow comments in core/paint

third_party/WebKit/Source/core/paint/PaintInvalidationCapableScrollableArea.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/paint/PaintInvalidationCapableScrollableArea.h"
 
 #include "core/frame/Settings.h"
 #include "core/html/HTMLFrameOwnerElement.h"
 #include "core/layout/LayoutBox.h"
 #include "core/layout/LayoutScrollbar.h"
@@ skipping 16 matching lines @@
             scrollbar, PaintInvalidationScroll);
 
   ScrollableArea::willRemoveScrollbar(scrollbar, orientation);
 }
 
 static LayoutRect scrollControlPaintInvalidationRect(
     const IntRect& scrollControlRect,
     const LayoutBox& box,
     const PaintInvalidatorContext& context) {
   LayoutRect paintInvalidationRect(scrollControlRect);
-  // No need to apply any paint offset. Scroll controls paint in a different transform space than their contained box
-  // (the scrollbarPaintOffset transform node).
+  // No need to apply any paint offset. Scroll controls paint in a different
+  // transform space than their contained box (the scrollbarPaintOffset
+  // transform node).
   if (!paintInvalidationRect.isEmpty() &&
       !RuntimeEnabledFeatures::slimmingPaintV2Enabled())
     context.mapLocalRectToPaintInvalidationBacking(box, paintInvalidationRect);
   return paintInvalidationRect;
 }
 
 // Returns true if the scroll control is invalidated.
 static bool invalidatePaintOfScrollControlIfNeeded(
     const LayoutRect& newPaintInvalidationRect,
     const LayoutRect& previousPaintInvalidationRect,
@@ skipping 19 matching lines @@
 static void invalidatePaintOfScrollbarIfNeeded(
     Scrollbar* scrollbar,
     GraphicsLayer* graphicsLayer,
     bool& previouslyWasOverlay,
     LayoutRect& previousPaintInvalidationRect,
     bool needsPaintInvalidationArg,
     LayoutBox& box,
     const PaintInvalidatorContext& context) {
   bool isOverlay = scrollbar && scrollbar->isOverlayScrollbar();
 
-  // Calculate paint invalidation rect of the scrollbar, except overlay composited scrollbars because we invalidate the graphics layer only.
+  // Calculate paint invalidation rect of the scrollbar, except overlay
+  // composited scrollbars because we invalidate the graphics layer only.
   LayoutRect newPaintInvalidationRect;
   if (scrollbar && !(graphicsLayer && isOverlay))
     newPaintInvalidationRect = scrollControlPaintInvalidationRect(
         scrollbar->frameRect(), box, context);
 
   bool needsPaintInvalidation = needsPaintInvalidationArg;
   if (needsPaintInvalidation && graphicsLayer) {
-    // If the scrollbar needs paint invalidation but didn't change location/size or the scrollbar is an
-    // overlay scrollbar (paint invalidation rect is empty), invalidating the graphics layer is enough
-    // (which has been done in ScrollableArea::setScrollbarNeedsPaintInvalidation()).
-    // Otherwise invalidatePaintOfScrollControlIfNeeded() below will invalidate the old and new location
-    // of the scrollbar on the box's paint invalidation container to ensure newly expanded/shrunk areas
-    // of the box to be invalidated.
+    // If the scrollbar needs paint invalidation but didn't change location/size
+    // or the scrollbar is an overlay scrollbar (paint invalidation rect is
+    // empty), invalidating the graphics layer is enough (which has been done in
+    // ScrollableArea::setScrollbarNeedsPaintInvalidation()).
+    // Otherwise invalidatePaintOfScrollControlIfNeeded() below will invalidate
+    // the old and new location of the scrollbar on the box's paint invalidation
+    // container to ensure newly expanded/shrunk areas of the box to be
+    // invalidated.
     needsPaintInvalidation = false;
     DCHECK(!graphicsLayer->drawsContent() ||
            graphicsLayer->getPaintController().cacheIsEmpty());
   }
 
-  // Invalidate the box's display item client if the box's padding box size is affected by change of the
-  // non-overlay scrollbar width. We detect change of paint invalidation rect size instead of change of
-  // scrollbar width change, which may have some false-positives (e.g. the scrollbar changed length but
-  // not width) but won't invalidate more than expected because in the false-positive case the box must
-  // have changed size and have been invalidated.
+  // Invalidate the box's display item client if the box's padding box size is
+  // affected by change of the non-overlay scrollbar width. We detect change of
+  // paint invalidation rect size instead of change of scrollbar width change,
+  // which may have some false-positives (e.g. the scrollbar changed length but
+  // not width) but won't invalidate more than expected because in the
+  // false-positive case the box must have changed size and have been
+  // invalidated.
   const LayoutBoxModelObject& paintInvalidationContainer =
       *context.paintInvalidationContainer;
   LayoutSize newScrollbarUsedSpaceInBox;
   if (!isOverlay)
     newScrollbarUsedSpaceInBox = newPaintInvalidationRect.size();
   LayoutSize previousScrollbarUsedSpaceInBox;
   if (!previouslyWasOverlay)
     previousScrollbarUsedSpaceInBox = previousPaintInvalidationRect.size();
   if (newScrollbarUsedSpaceInBox != previousScrollbarUsedSpaceInBox) {
     context.paintingLayer->setNeedsRepaint();
@@ skipping 80 matching lines @@
   fullBounds.unite(m_scrollCornerAndResizerPreviousPaintInvalidationRect);
   return fullBounds;
 }
 
 void PaintInvalidationCapableScrollableArea::
     scrollControlWasSetNeedsPaintInvalidation() {
   layoutBox()->setMayNeedPaintInvalidation();
 }
 
 }  // namespace blink