[SPInvalidation] Handle pixel-snapping of paint invalidation rects

third_party/WebKit/Source/core/paint/ObjectPaintInvalidator.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/ObjectPaintInvalidator.h"
 
 #include "core/frame/FrameView.h"
 #include "core/frame/LocalFrame.h"
 #include "core/layout/LayoutBlockFlow.h"
 #include "core/layout/LayoutView.h"
@@ skipping 393 matching lines @@
     backgroundObscurationChanged = true;
   }
 
   if (m_context.forcedSubtreeInvalidationFlags &
       PaintInvalidatorContext::ForcedSubtreeFullInvalidation)
     return PaintInvalidationSubtree;
 
   if (m_object.shouldDoFullPaintInvalidation())
     return m_object.fullPaintInvalidationReason();
 
-  if (m_context.oldBounds.isEmpty() && m_context.newBounds.isEmpty())
+  if (m_context.oldBounds.rect.isEmpty() && m_context.newBounds.rect.isEmpty())
     return PaintInvalidationNone;
 
   if (backgroundObscurationChanged)
     return PaintInvalidationBackgroundObscurationChange;
 
   if (m_object.paintedOutputOfObjectHasNoEffectRegardlessOfSize())
     return PaintInvalidationNone;
 
   const ComputedStyle& style = m_object.styleRef();
 
   // The outline may change shape because of position change of descendants. For
   // simplicity, just force full paint invalidation if this object is marked for
   // checking paint invalidation for any reason.
   // TODO(wangxianzhu): Optimize this.
   if (style.hasOutline())
     return PaintInvalidationOutline;
 
   // If the size is zero on one of our bounds then we know we're going to have
   // to do a full invalidation of either old bounds or new bounds.
-  if (m_context.oldBounds.isEmpty())
+  if (m_context.oldBounds.rect.isEmpty())
     return PaintInvalidationBecameVisible;
-  if (m_context.newBounds.isEmpty())
+  if (m_context.newBounds.rect.isEmpty())
     return PaintInvalidationBecameInvisible;
 
   // If we shifted, we don't know the exact reason so we are conservative and
   // trigger a full invalidation. Shifting could be caused by some layout
   // property (left / top) or some in-flow layoutObject inserted / removed
   // before us in the tree.
-  if (m_context.newBounds.location() != m_context.oldBounds.location())
+  if (m_context.newBounds.rect.location() !=
+      m_context.oldBounds.rect.location())
     return PaintInvalidationBoundsChange;
 
   if (m_context.newLocation != m_context.oldLocation)
     return PaintInvalidationLocationChange;
 
   // Incremental invalidation is only applicable to LayoutBoxes. Return
   // PaintInvalidationIncremental no matter if oldBounds and newBounds are equal
   // because a LayoutBox may need paint invalidation if its border box changes.
   if (m_object.isBox())
     return PaintInvalidationIncremental;
 
-  if (m_context.oldBounds != m_context.newBounds)
+  if (m_context.oldBounds.rect != m_context.newBounds.rect)
     return PaintInvalidationBoundsChange;
 
   return PaintInvalidationNone;
 }
 
 void ObjectPaintInvalidatorWithContext::invalidateSelectionIfNeeded(
     PaintInvalidationReason reason) {
   // Update selection rect when we are doing full invalidation (in case that the
   // object is moved, composite status changed, etc.) or
   // shouldInvalidationSelection is set (in case that the selection itself
@@ skipping 40 matching lines @@
       if (m_context.forcedSubtreeInvalidationFlags &
           PaintInvalidatorContext::ForcedSubtreeInvalidationChecking) {
         reason = PaintInvalidationLocationChange;
         break;
       }
       return PaintInvalidationNone;
     case PaintInvalidationDelayedFull:
       return PaintInvalidationDelayedFull;
     default:
       DCHECK(isImmediateFullPaintInvalidationReason(reason));
-      fullyInvalidatePaint(reason, m_context.oldBounds, m_context.newBounds);
+      fullyInvalidatePaint(reason, m_context.oldBounds.rect,
+                           m_context.newBounds.rect);
   }
 
   m_context.paintingLayer->setNeedsRepaint();
   m_object.invalidateDisplayItemClients(reason);
   return reason;
 }
 
 DisablePaintInvalidationStateAsserts::DisablePaintInvalidationStateAsserts()
     : m_disabler(&gDisablePaintInvalidationStateAsserts, true) {}
 
 }  // namespace blink