Fix clip and offset issues of slimmingPaintInvalidation for SPv1

third_party/WebKit/Source/core/paint/PaintInvalidator.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/PaintInvalidator.h"
 
 #include "core/editing/FrameSelection.h"
 #include "core/frame/FrameView.h"
 #include "core/frame/LocalFrame.h"
 #include "core/frame/Settings.h"
@@ skipping 17 matching lines @@
     // (see LayoutBoxModelObject.h) but we need them to be in physical
     // coordinates.
     FloatRect rect = localRect;
     if (object.isBox())
         toLayoutBox(object).flipForWritingMode(rect);
 
     LayoutRect result;
     if (object == context.paintInvalidationContainer) {
         result = LayoutRect(rect);
     } else {
-        rect.moveBy(FloatPoint(context.treeBuilderContext.current.paintOffset));
-
-        bool success = false;
         PropertyTreeState currentTreeState(context.treeBuilderContext.current.transform, context.treeBuilderContext.current.clip, context.treeBuilderContext.currentEffect);
         PropertyTreeState containerTreeState;
-        context.paintInvalidationContainer->objectPaintProperties()->getContentsProperties(containerTreeState);
+        const ObjectPaintProperties* containerPaintProperties = context.paintInvalidationContainer->objectPaintProperties();
+        containerPaintProperties->getContentsProperties(containerTreeState);
+
+        rect.moveBy(FloatPoint(context.treeBuilderContext.current.paintOffset));
+        bool success = false;
         result = LayoutRect(geometryMapper.mapToVisualRectInDestinationSpace(rect, currentTreeState, containerTreeState, success));
         DCHECK(success);
+        result.moveBy(-containerPaintProperties->localBorderBoxProperties()->paintOffset);
     }
 
     if (context.paintInvalidationContainer->layer()->groupedMapping())
         PaintLayer::mapRectInPaintInvalidationContainerToBacking(*context.paintInvalidationContainer, result);
     return result;
 }
 
 void PaintInvalidatorContext::mapLocalRectToPaintInvalidationBacking(const LayoutObject& object, LayoutRect& rect) const
 {
     GeometryMapper geometryMapper;
@@ skipping 132 matching lines @@
     if (frameView.frame().selection().isCaretBoundsDirty())
         frameView.frame().selection().invalidateCaretRect();
 
     // Temporary callback for crbug.com/487345,402044
     // TODO(ojan): Make this more general to be used by PositionObserver
     // and rAF throttling.
     IntRect visibleRect = frameView.rootFrameToContents(frameView.computeVisibleArea());
     layoutView->sendMediaPositionChangeNotifications(visibleRect);
 }
 
+static bool hasPercentageTransform(const ComputedStyle& style)
+{
+    if (TransformOperation* translate = style.translate()) {
+        if (translate->dependsOnBoxSize())
+            return true;
+    }
+    return style.transform().dependsOnBoxSize()
+        || (style.transformOriginX() != Length(50, Percent) && style.transformOriginX().isPercentOrCalc())
+        || (style.transformOriginY() != Length(50, Percent) && style.transformOriginY().isPercentOrCalc());
+}
+

[Xianzhu, 2016/09/13 17:42:33]

This and the changes below are copied from
LayoutBoxMobelObject::invalidateTreeIfNeeded(). 

 void PaintInvalidator::invalidatePaintIfNeeded(const LayoutObject& object, PaintInvalidatorContext& context)
 {
     object.getMutableForPainting().ensureIsReadyForPaintInvalidation();
 
     if (!context.forcedSubtreeInvalidationFlags && !object.shouldCheckForPaintInvalidationRegardlessOfPaintInvalidationState())
         return;
 
     updatePaintingLayer(object, context);
 
     if (object.document().printing())
         return; // Don't invalidate paints if we're printing.
 
     TRACE_EVENT1(TRACE_DISABLED_BY_DEFAULT("blink.invalidation"), "PaintInvalidator::invalidatePaintIfNeeded()", "object", object.debugName().ascii());
 
     updateContext(object, context);
 
     if (!object.shouldCheckForPaintInvalidationRegardlessOfPaintInvalidationState() && context.forcedSubtreeInvalidationFlags == PaintInvalidatorContext::ForcedSubtreeInvalidationRectUpdate) {
         // We are done updating the paint invalidation rect. No other paint invalidation work to do for this object.
         return;
     }
 
-    switch (object.invalidatePaintIfNeeded(context)) {
+    PaintInvalidationReason reason = object.invalidatePaintIfNeeded(context);
+    switch (reason) {
     case PaintInvalidationDelayedFull:
         m_pendingDelayedPaintInvalidations.append(&object);
         break;
     case PaintInvalidationSubtree:
         context.forcedSubtreeInvalidationFlags |= (PaintInvalidatorContext::ForcedSubtreeFullInvalidation | PaintInvalidatorContext::ForcedSubtreeFullInvalidationForStackedContents);
         break;
     case PaintInvalidationSVGResourceChange:
         context.forcedSubtreeInvalidationFlags |= PaintInvalidatorContext::ForcedSubtreeInvalidationChecking;
         break;
     default:
         break;
     }
 
     if (context.oldLocation != context.newLocation)
         context.forcedSubtreeInvalidationFlags |= PaintInvalidatorContext::ForcedSubtreeInvalidationChecking;
 
+    // TODO(crbug.com/533277): This is a workaround for the bug. Remove when we detect paint offset change.
+    if (reason != PaintInvalidationNone && hasPercentageTransform(object.styleRef()))
+        context.forcedSubtreeInvalidationFlags |= PaintInvalidatorContext::ForcedSubtreeInvalidationChecking;
+
+    // TODO(crbug.com/490725): This is a workaround for the bug, to force descendant to update paint invalidation
+    // rects on clipping change.
+    if (!RuntimeEnabledFeatures::slimmingPaintV2Enabled()
+        && context.oldBounds != context.newBounds
+        // Note that isLayoutView() below becomes unnecessary after the launch of root layer scrolling.
+        && (object.hasOverflowClip() || object.isLayoutView())
+        && !toLayoutBox(object).usesCompositedScrolling())
+        context.forcedSubtreeInvalidationFlags |= PaintInvalidatorContext::ForcedSubtreeInvalidationRectUpdate;
+
     object.getMutableForPainting().clearPaintInvalidationFlags();
 }
 
 void PaintInvalidator::processPendingDelayedPaintInvalidations()
 {
     for (auto target : m_pendingDelayedPaintInvalidations)
         target->getMutableForPainting().setShouldDoFullPaintInvalidation(PaintInvalidationDelayedFull);
 }
 
 } // namespace blink