Handle border-image with fill and zero-width borders

Source/core/layout/LayoutObject.cpp

Index: Source/core/layout/LayoutObject.cpp
diff --git a/Source/core/layout/LayoutObject.cpp b/Source/core/layout/LayoutObject.cpp
index f668620b095015648d489dd76352b72f812f59c8..306fccde90b11b663dcd9c15ea4222a3c1286940 100644
--- a/Source/core/layout/LayoutObject.cpp
+++ b/Source/core/layout/LayoutObject.cpp
@@ -689,7 +689,7 @@ LayoutFlowThread* LayoutObject::locateFlowThreadContainingBlock() const
 // FIXME: This is incorrect for document element. Remove this when we enable slimming paint.
 static inline bool layoutObjectHasNoBoxEffectObsolete(const LayoutObject& object)
 {
-    return !object.style()->hasVisualOverflowingEffect() && !object.style()->hasBorder() && !object.style()->hasBackground();
+    return !object.style()->hasVisualOverflowingEffect() && !object.style()->hasBorderDecoration() && !object.style()->hasBackground();
 }
 
 bool LayoutObject::skipInvalidationWhenLaidOutChildren() const
@@ -1668,7 +1668,7 @@ StyleDifference LayoutObject::adjustStyleDifference(StyleDifference diff) const
     }
 
     if (diff.textOrColorChanged() && !diff.needsPaintInvalidation()) {
-        if (style()->hasBorder() || style()->hasOutline()
+        if (style()->hasBorderDecoration() || style()->hasOutline()

[pdr., 2015/06/02 03:07:56]

I think this may be correct as just hasBorder because the decoration won't
depend on the color. If the color property changes we need to check if a border
exists because border-color can use the 'color' due to css weirdism.

[davve, 2015/06/03 06:56:01]

Ah, sounds reasonable. Thanks.

             || (isText() && !toLayoutText(this)->isAllCollapsibleWhitespace()))
             diff.setNeedsPaintInvalidationObject();
     }