Replace coversExtraPixels with simpler logic

third_party/WebKit/Source/core/paint/BoxPaintInvalidator.cpp

 // Copyright 2016 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/BoxPaintInvalidator.h"
 
 #include "core/frame/Settings.h"
 #include "core/layout/LayoutView.h"
 #include "core/paint/ObjectPaintInvalidator.h"
 #include "core/paint/PaintInvalidator.h"
@@ skipping 47 matching lines @@
   }
   if (delta < 0) {
     return LayoutRect(location.x(),
                       location.y() + newSize.height() - extraHeight,
                       oldSize.width(), -delta + extraHeight);
   }
   return LayoutRect();
 }
 
 bool BoxPaintInvalidator::incrementallyInvalidatePaint() {
-  const LayoutRect& oldRect = m_context.oldBounds.rect;
-  const LayoutRect& newRect = m_context.newBounds.rect;
-  DCHECK(oldRect.location() == newRect.location());
-  LayoutRect rightDelta =
-      computeRightDelta(newRect.location(), oldRect.size(), newRect.size(), 0);
-  LayoutRect bottomDelta =
-      computeBottomDelta(newRect.location(), oldRect.size(), newRect.size(), 0);
-
-  if (m_box.styleRef().hasBorder() || m_box.styleRef().hasBackground()) {
-    LayoutSize oldBorderBoxSize = computePreviousBorderBoxSize(oldRect.size());
+  LayoutRect rightDelta;
+  LayoutRect bottomDelta;
+  if (m_box.isLayoutView()) {
+    // This corresponds to the special case in computePaintInvalidationReason()
+    // for LayoutView in non-rootLayerScrolling mode. In rootLayerScrolling
+    // mode, we'll do full paint invalidation (see crbug.com/660156).
+    DCHECK(!RuntimeEnabledFeatures::rootLayerScrollingEnabled());
+    DCHECK(m_context.oldBounds.location() == m_context.newBounds.location());
+    rightDelta = computeRightDelta(m_context.newBounds.location(),
+                                   m_context.oldBounds.size(),
+                                   m_context.newBounds.size(), 0);
+    bottomDelta = computeBottomDelta(m_context.newBounds.location(),
+                                     m_context.oldBounds.size(),
+                                     m_context.newBounds.size(), 0);
+  } else {
+    LayoutSize oldBorderBoxSize =
+        computePreviousBorderBoxSize(m_context.oldBounds.size());
     LayoutSize newBorderBoxSize = m_box.size();
     DCHECK(m_context.oldLocation == m_context.newLocation);
-    rightDelta.unite(computeRightDelta(m_context.newLocation, oldBorderBoxSize,
-                                       newBorderBoxSize, m_box.borderRight()));
-    bottomDelta.unite(computeBottomDelta(m_context.newLocation,
-                                         oldBorderBoxSize, newBorderBoxSize,
-                                         m_box.borderBottom()));
+    rightDelta = computeRightDelta(m_context.newLocation, oldBorderBoxSize,
+                                   newBorderBoxSize, m_box.borderRight());
+    bottomDelta = computeBottomDelta(m_context.newLocation, oldBorderBoxSize,
+                                     newBorderBoxSize, m_box.borderBottom());
   }
 
   if (rightDelta.isEmpty() && bottomDelta.isEmpty())
     return false;
 
-  invalidatePaintRectClippedByOldAndNewBounds(rightDelta);
-  invalidatePaintRectClippedByOldAndNewBounds(bottomDelta);
+  ObjectPaintInvalidator objectPaintInvalidator(m_box);
+  objectPaintInvalidator.invalidatePaintUsingContainer(
+      *m_context.paintInvalidationContainer, rightDelta,
+      PaintInvalidationIncremental);
+  objectPaintInvalidator.invalidatePaintUsingContainer(
+      *m_context.paintInvalidationContainer, bottomDelta,
+      PaintInvalidationIncremental);
   return true;
 }
 
-void BoxPaintInvalidator::invalidatePaintRectClippedByOldAndNewBounds(

[Xianzhu, 2016/10/28 22:03:06]

Now this is not needed because we incrementally invalidate only when the bounds
equals translated border box. Previously this excludes clipped overflows from
the paint invalidation rects.

-    const LayoutRect& rect) {
-  if (rect.isEmpty())
-    return;
-
-  ObjectPaintInvalidator objectPaintInvalidator(m_box);
-  LayoutRect rectClippedByOldBounds =
-      intersection(rect, m_context.oldBounds.rect);
-  LayoutRect rectClippedByNewBounds =
-      intersection(rect, m_context.newBounds.rect);
-  // Invalidate only once if the clipped rects equal.
-  if (rectClippedByOldBounds == rectClippedByNewBounds) {
-    objectPaintInvalidator.invalidatePaintUsingContainer(
-        *m_context.paintInvalidationContainer, rectClippedByOldBounds,
-        PaintInvalidationIncremental);
-    return;
-  }
-  // Invalidate the bigger one if one contains another. Otherwise invalidate
-  // both.
-  if (!rectClippedByNewBounds.contains(rectClippedByOldBounds))
-    objectPaintInvalidator.invalidatePaintUsingContainer(
-        *m_context.paintInvalidationContainer, rectClippedByOldBounds,
-        PaintInvalidationIncremental);
-  if (!rectClippedByOldBounds.contains(rectClippedByNewBounds))
-    objectPaintInvalidator.invalidatePaintUsingContainer(
-        *m_context.paintInvalidationContainer, rectClippedByNewBounds,
-        PaintInvalidationIncremental);
-}
-
 PaintInvalidationReason BoxPaintInvalidator::computePaintInvalidationReason() {
   PaintInvalidationReason reason =
       ObjectPaintInvalidatorWithContext(m_box, m_context)
           .computePaintInvalidationReason();
 
   if (isImmediateFullPaintInvalidationReason(reason) ||
       reason == PaintInvalidationNone)
     return reason;
 
   if (m_box.mayNeedPaintInvalidationAnimatedBackgroundImage() &&
       !m_box.backgroundIsKnownToBeObscured())
     reason = PaintInvalidationDelayedFull;
 
   // If the current paint invalidation reason is PaintInvalidationDelayedFull,
   // then this paint invalidation can delayed if the LayoutBox in question is
   // not on-screen. The logic to decide whether this is appropriate exists at
   // the site of the original paint invalidation that chose
   // PaintInvalidationDelayedFull.
   if (reason == PaintInvalidationDelayedFull) {
     // Do regular full paint invalidation if the object is onscreen.
     return m_box.intersectsVisibleViewport() ? PaintInvalidationFull
                                              : PaintInvalidationDelayedFull;
   }
 
+  DCHECK(reason == PaintInvalidationIncremental);
+
   if (m_box.isLayoutView()) {
     const LayoutView& layoutView = toLayoutView(m_box);
     // In normal compositing mode, root background doesn't need to be
     // invalidated for box changes, because the background always covers the
     // whole document rect and clipping is done by
     // compositor()->m_containerLayer. Also the scrollbars are always
     // composited. There are no other box decoration on the LayoutView thus we
     // can safely exit here.
     if (layoutView.usesCompositing() &&
         !RuntimeEnabledFeatures::rootLayerScrollingEnabled())
       return reason;
   }
 
-  if (reason == PaintInvalidationIncremental &&
-      m_context.oldBounds.rect != m_context.newBounds.rect) {
-    if (m_context.newBounds.coversExtraPixels ||
-        m_context.oldBounds.coversExtraPixels) {
-      // Incremental invalidation is not applicable because the difference
-      // between oldBounds and newBounds may not cover all changed pixels along
-      // the edges.
-      return PaintInvalidationBoundsChange;
-    }
-
-    // If the transform is not identity or translation, incremental invalidation
-    // is not applicable because the difference between oldBounds and newBounds
-    // doesn't cover all area needing invalidation.
-    // TODO(crbug.com/426111): Should also consider ancestor transforms
-    // since paintInvalidationContainer. Combine this logic into the above
-    // boundsCoversExtraPixels logic.
-    if (m_context.paintInvalidationContainer != m_box && m_box.hasLayer() &&
-        m_box.layer()->transform() &&
-        !m_box.layer()->transform()->isIdentityOrTranslation())
-      return PaintInvalidationBoundsChange;
-  }
-
   const ComputedStyle& style = m_box.styleRef();
   if (style.backgroundLayers().thisOrNextLayersUseContentBox() ||
       style.maskLayers().thisOrNextLayersUseContentBox() ||
       style.boxSizing() == BoxSizingBorderBox) {
     if (previousBoxSizesMap().get(&m_box).contentBoxRect !=
         m_box.contentBoxRect())
       return PaintInvalidationContentBoxChange;
   }
 
-  if (!style.hasBackground() && !style.hasBoxDecorations()) {
-    if (reason == PaintInvalidationIncremental &&
-        m_context.oldBounds.rect != m_context.newBounds.rect &&
-        m_box.hasNonCompositedScrollbars())
-      return PaintInvalidationBorderBoxChange;
-    return reason;
-  }
-
   if (style.backgroundLayers().thisOrNextLayersHaveLocalAttachment()) {
     if (previousBoxSizesMap().get(&m_box).layoutOverflowRect !=
         m_box.layoutOverflowRect())
       return PaintInvalidationLayoutOverflowBoxChange;
   }
 
   LayoutSize oldBorderBoxSize =
-      computePreviousBorderBoxSize(m_context.oldBounds.rect.size());
+      computePreviousBorderBoxSize(m_context.oldBounds.size());
   LayoutSize newBorderBoxSize = m_box.size();
+  bool borderBoxChanged = oldBorderBoxSize != newBorderBoxSize;
 
-  if (oldBorderBoxSize == newBorderBoxSize)
-    return reason;
+  if (!borderBoxChanged && m_context.oldBounds == m_context.newBounds)
+    return PaintInvalidationNone;
 
-  // See another hasNonCompositedScrollbars() callsite above.
+  // If either border box changed or bounds changed, and old or new border box
+  // doesn't equal old or new bounds, incremental invalidation is not
+  // applicable.
+  if (m_context.oldBounds !=
+          LayoutRect(m_context.oldLocation, oldBorderBoxSize) ||
+      m_context.newBounds !=
+          LayoutRect(m_context.newLocation, newBorderBoxSize)) {
+    return borderBoxChanged ? PaintInvalidationBorderBoxChange
+                            : PaintInvalidationBoundsChange;
+  }
+
+  DCHECK(borderBoxChanged);
+
   if (m_box.hasNonCompositedScrollbars())
     return PaintInvalidationBorderBoxChange;
 
   if (style.hasVisualOverflowingEffect() || style.hasAppearance() ||
       style.hasFilterInducingProperty() || style.resize() != RESIZE_NONE ||
       style.hasMask())
     return PaintInvalidationBorderBoxChange;
 
   if (style.hasBorderRadius())
     return PaintInvalidationBorderBoxChange;
 
   if (oldBorderBoxSize.width() != newBorderBoxSize.width() &&
       m_box.mustInvalidateBackgroundOrBorderPaintOnWidthChange())
     return PaintInvalidationBorderBoxChange;
   if (oldBorderBoxSize.height() != newBorderBoxSize.height() &&
       m_box.mustInvalidateBackgroundOrBorderPaintOnHeightChange())
     return PaintInvalidationBorderBoxChange;
 
-  return reason;
+  return PaintInvalidationIncremental;
 }
 
 PaintInvalidationReason BoxPaintInvalidator::invalidatePaintIfNeeded() {
   PaintInvalidationReason reason = computePaintInvalidationReason();
   if (reason == PaintInvalidationIncremental) {
     if (incrementallyInvalidatePaint()) {
       m_context.paintingLayer->setNeedsRepaint();
       m_box.invalidateDisplayItemClients(reason);
     } else {
       reason = PaintInvalidationNone;
@@ skipping 13 matching lines @@
   if (PaintLayerScrollableArea* area = m_box.getScrollableArea())
     area->invalidatePaintOfScrollControlsIfNeeded(m_context);
 
   // This is for the next invalidatePaintIfNeeded so must be at the end.
   savePreviousBoxSizesIfNeeded();
 
   return reason;
 }
 
 bool BoxPaintInvalidator::needsToSavePreviousBoxSizes() {
-  LayoutSize paintInvalidationSize = m_context.newBounds.rect.size();
+  LayoutSize paintInvalidationSize = m_context.newBounds.size();
   // Don't save old box sizes if the paint rect is empty because we'll
   // full invalidate once the paint rect becomes non-empty.
   if (paintInvalidationSize.isEmpty())
     return false;
 
+  if (m_box.paintedOutputOfObjectHasNoEffectRegardlessOfSize())
+    return false;
+
   const ComputedStyle& style = m_box.styleRef();
 
   // If we use border-box sizing we need to track changes in the size of the
   // content box.
   if (style.boxSizing() == BoxSizingBorderBox)
     return true;
 
-  // We need the old box sizes only when the box has background, decorations, or
-  // masks.
-  // Main LayoutView paints base background, thus interested in box size.
-  if (!m_box.isLayoutView() && !style.hasBackground() &&
-      !style.hasBoxDecorations() && !style.hasMask())
-    return false;
-
   // No need to save old border box size if we can use size of the old paint
   // rect as the old border box size in the next invalidation.
   if (paintInvalidationSize != m_box.size())
     return true;
 
   // Background and mask layers can depend on other boxes than border box. See
   // crbug.com/490533
   if (style.backgroundLayers().thisOrNextLayersUseContentBox() ||
       style.backgroundLayers().thisOrNextLayersHaveLocalAttachment() ||
       style.maskLayers().thisOrNextLayersUseContentBox())
     return true;
 
   return false;
 }
 
 void BoxPaintInvalidator::savePreviousBoxSizesIfNeeded() {
   if (!needsToSavePreviousBoxSizes()) {
     previousBoxSizesMap().remove(&m_box);
     return;
   }
 
   PreviousBoxSizes sizes = {m_box.size(), m_box.contentBoxRect(),
                             m_box.layoutOverflowRect()};
   previousBoxSizesMap().set(&m_box, sizes);
 }
 
 LayoutSize BoxPaintInvalidator::computePreviousBorderBoxSize(
     const LayoutSize& previousBoundsSize) {
-  // PreviousBorderBoxSize is only valid when there is background or box
-  // decorations.
-  DCHECK(m_box.styleRef().hasBackground() ||
-         m_box.styleRef().hasBoxDecorations());
-
   auto it = previousBoxSizesMap().find(&m_box);
   if (it != previousBoxSizesMap().end())
     return it->value.borderBoxSize;
 
   // We didn't save the old border box size because it was the same as the size
   // of oldBounds.
   return previousBoundsSize;
 }
 
 }  // namespace blink