[SPInvalidation] Update paint properties when border radii change

third_party/WebKit/Source/core/layout/LayoutBox.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com)
  *           (C) 2005, 2006 Samuel Weinig (sam.weinig@gmail.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc.
  *               All rights reserved.
  * Copyright (C) 2013 Adobe Systems Incorporated. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 305 matching lines @@
     placeholder->layoutObjectInFlowThreadStyleDidChange(oldStyle);
 
   updateBackgroundAttachmentFixedStatusAfterStyleChange();
 
   if (oldStyle) {
     LayoutFlowThread* flowThread = flowThreadContainingBlock();
     if (flowThread && flowThread != this)
       flowThread->flowThreadDescendantStyleDidChange(this, diff, *oldStyle);
 
     updateScrollSnapMappingAfterStyleChange(&newStyle, oldStyle);
-  }
 
-  if (RuntimeEnabledFeatures::slimmingPaintInvalidationEnabled()) {
-    if (hasOverflowClip() || styleRef().containsPaint() || hasControlClip()) {
+    if (RuntimeEnabledFeatures::slimmingPaintInvalidationEnabled() &&
+        shouldClipOverflow()) {
       // The overflow clip paint property depends on border sizes through
-      // overflowClipRect() so we update properties on border size changes.
-      if (oldStyle && !oldStyle->border().sizeEquals(newStyle.border()))
+      // overflowClipRect(), and border radii, so we update properties on
+      // border size or radii change.
+      if (!oldStyle->border().sizeEquals(newStyle.border()) ||
+          !oldStyle->border().radiiEqual(newStyle.border()))
         setNeedsPaintPropertyUpdate();
     }
   }
 
   // Non-atomic inlines should be LayoutInline or LayoutText, not LayoutBox.
   DCHECK(!isInline() || isAtomicInlineLevel());
 }
 
 void LayoutBox::updateBackgroundAttachmentFixedStatusAfterStyleChange() {
   if (!frameView())
@@ skipping 1352 matching lines @@
 void LayoutBox::frameRectChanged() {
   if (node() && node()->isElementNode()) {
     Element& element = toElement(*node());
     element.setNeedsResizeObserverUpdate();
   }
   // The frame rect may change because of layout of other objects.
   // Should check this object for paint invalidation.
   if (!needsLayout())
     setMayNeedPaintInvalidation();
 
-  if (RuntimeEnabledFeatures::slimmingPaintInvalidationEnabled()) {
-    // The overflow clip paint property depends on the border box rect through
-    // overflowClipRect(). The border box rect's size equals the frame rect's
-    // size, so we trigger a paint property update when the framerect changes.
-    if (hasOverflowClip() || styleRef().containsPaint() || hasControlClip())
-      setNeedsPaintPropertyUpdate();
-  }
+  // The overflow clip paint property depends on the border box rect through
+  // overflowClipRect(). The border box rect's size equals the frame rect's
+  // size, so we trigger a paint property update when the framerect changes.
+  if (RuntimeEnabledFeatures::slimmingPaintInvalidationEnabled() &&
+      shouldClipOverflow())
+    setNeedsPaintPropertyUpdate();
 }
 
 bool LayoutBox::intersectsVisibleViewport() const {
   LayoutRect rect = visualOverflowRect();
   LayoutView* layoutView = view();
   while (!layoutView->frame()->ownerLayoutItem().isNull())
     layoutView =
         LayoutAPIShim::layoutObjectFrom(layoutView->frame()->ownerLayoutItem())
             ->view();
   mapToVisualRectInAncestorSpace(layoutView, rect);
@@ skipping 3940 matching lines @@
 LayoutRect LayoutBox::debugRect() const {
   LayoutRect rect = frameRect();
 
   LayoutBlock* block = containingBlock();
   if (block)
     block->adjustChildDebugRect(rect);
 
   return rect;
 }
 
+bool LayoutBox::shouldClipOverflow() const {
+  return hasOverflowClip() || styleRef().containsPaint() || hasControlClip();
+}
+
 }  // namespace blink