Fix under-invalidation on SVG shape/image resize in subpixels

third_party/WebKit/Source/core/layout/svg/LayoutSVGImage.cpp

 /*
  * Copyright (C) 2006 Alexander Kellett <lypanov@kde.org>
  * Copyright (C) 2006 Apple Computer, Inc.
  * Copyright (C) 2007 Nikolas Zimmermann <zimmermann@kde.org>
  * Copyright (C) 2007, 2008, 2009 Rob Buis <buis@kde.org>
  * Copyright (C) 2009 Google, Inc.
  * Copyright (C) 2009 Dirk Schulze <krit@webkit.org>
  * Copyright (C) 2010 Patrick Gansterer <paroga@paroga.com>
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 70 matching lines @@
         m_objectBoundingBox.width(),
         resolveHeightForRatio(m_objectBoundingBox.width(), intrinsicSize));
 
   DCHECK(styleRef().width().isAuto());
   return FloatSize(
       resolveWidthForRatio(m_objectBoundingBox.height(), intrinsicSize),
       m_objectBoundingBox.height());
 }
 
 bool LayoutSVGImage::updateBoundingBox() {
-  FloatRect oldBoundaries = m_objectBoundingBox;
+  FloatRect oldObjectBoundingBox = m_objectBoundingBox;
 
   SVGLengthContext lengthContext(element());
   m_objectBoundingBox =
       FloatRect(lengthContext.valueForLength(styleRef().svgStyle().x(),
                                              styleRef(), SVGLengthMode::Width),
                 lengthContext.valueForLength(styleRef().svgStyle().y(),
                                              styleRef(), SVGLengthMode::Height),
                 lengthContext.valueForLength(styleRef().width(), styleRef(),
                                              SVGLengthMode::Width),
                 lengthContext.valueForLength(styleRef().height(), styleRef(),
                                              SVGLengthMode::Height));
 
   if (styleRef().width().isAuto() || styleRef().height().isAuto())
     m_objectBoundingBox.setSize(calculateObjectSize());
 
-  m_needsBoundariesUpdate |= oldBoundaries != m_objectBoundingBox;
+  if (oldObjectBoundingBox != m_objectBoundingBox) {
+    setShouldDoFullPaintInvalidation();
+    m_needsBoundariesUpdate = true;
+  }
+
   if (element())
     element()->setNeedsResizeObserverUpdate();
 
-  return oldBoundaries.size() != m_objectBoundingBox.size();
+  return oldObjectBoundingBox.size() != m_objectBoundingBox.size();
 }
 
 void LayoutSVGImage::layout() {
   ASSERT(needsLayout());
   LayoutAnalyzer::Scope analyzer(*this);
 
   // Invalidate all resources of this client if our layout changed.
   if (everHadLayout() && selfNeedsLayout())
     SVGResourcesCache::clientLayoutChanged(this);
 
@@ skipping 77 matching lines @@
 
 void LayoutSVGImage::addOutlineRects(Vector<LayoutRect>& rects,
                                      const LayoutPoint&,
                                      IncludeBlockVisualOverflowOrNot) const {
   // this is called from paint() after the localTransform has already been
   // applied
   rects.append(LayoutRect(paintInvalidationRectInLocalSVGCoordinates()));
 }
 
 }  // namespace blink