Handle border-image with fill and zero-width borders

Source/core/layout/LayoutObject.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Dirk Mueller (mueller@kde.org)
  *           (C) 2004 Allan Sandfeld Jensen (kde@carewolf.com)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2009 Google Inc. All rights reserved.
  * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 671 matching lines @@
             curr = curr->parent();
         }
     }
     return nullptr;
 }
 
 // FIXME: This could be used when changing the size of a layoutObject without children to skip some invalidations.
 // 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
 {
     if (!neededLayoutBecauseOfChildren())
         return false;
 
     // SVG layoutObjects need to be invalidated when their children are laid out.
     // LayoutBlocks with line boxes are responsible to invalidate them so we can't ignore them.
     if (isSVG() || (isLayoutBlockFlow() && toLayoutBlockFlow(this)->firstLineBox()))
@@ skipping 958 matching lines @@
     }
 
     // If filter changed, and the layer does not paint into its own separate backing or it paints with filters, then we need to invalidate paints.
     if (diff.filterChanged() && hasLayer()) {
         DeprecatedPaintLayer* layer = toLayoutBoxModelObject(this)->layer();
         if (!layer->hasStyleDeterminedDirectCompositingReasons() || layer->paintsWithFilters())
             diff.setNeedsPaintInvalidationLayer();
     }
 
     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();
     }
 
     // The answer to layerTypeRequired() for plugins, iframes, and canvas can change without the actual
     // style changing, since it depends on whether we decide to composite these elements. When the
     // layer status of one of these elements changes, we need to force a layout.
     if (!diff.needsFullLayout() && style() && isLayoutBoxModelObject()) {
         bool requiresLayer = toLayoutBoxModelObject(this)->layerTypeRequired() != NoDeprecatedPaintLayer;
         if (hasLayer() != requiresLayer)
@@ skipping 1618 matching lines @@
         const blink::LayoutObject* root = object1;
         while (root->parent())
             root = root->parent();
         root->showLayoutTreeAndMark(object1, "*", object2, "-", 0);
     } else {
         fprintf(stderr, "Cannot showLayoutTree. Root is (nil)\n");
     }
 }
 
 #endif