Add UseCounter for border-image overriding border-widths

third_party/WebKit/Source/core/paint/NinePieceImagePainter.cpp

 // Copyright 2015 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 "config.h"
 #include "core/paint/NinePieceImagePainter.h"
 
+#include "core/frame/UseCounter.h"
 #include "core/layout/ImageQualityController.h"
 #include "core/layout/LayoutBoxModelObject.h"
 #include "core/paint/BoxPainter.h"
 #include "core/paint/NinePieceImageGrid.h"
 #include "core/style/ComputedStyle.h"
 #include "core/style/NinePieceImage.h"
 #include "platform/geometry/IntSize.h"
 #include "platform/geometry/LayoutRect.h"
 #include "platform/graphics/GraphicsContext.h"
 
@@ skipping 10 matching lines @@
     StyleImage* styleImage = ninePieceImage.image();
     if (!styleImage)
         return false;
 
     if (!styleImage->isLoaded())
         return true; // Never paint a nine-piece image incrementally, but don't paint the fallback borders either.
 
     if (!styleImage->canRender(m_layoutObject, style.effectiveZoom()))
         return false;
 
+    if ((style.borderLeftWidth() && style.borderLeft().style() == BNONE)

[philipj_slow, 2015/11/18 09:28:58]

Comment that these conditions are meant to isolate whether the
!m_image.hasImage() part of borderLeftWidth() had any effect, as that's the only
way that border-style:none does not imply border-width:0.

Also, should BHIDDEN be included here?

+        || (style.borderRightWidth() && style.borderRight().style() == BNONE)
+        || (style.borderTopWidth() && style.borderTop().style() == BNONE)
+        || (style.borderBottomWidth() && style.borderBottom().style() == BNONE))
+        UseCounter::count(m_layoutObject.document(), UseCounter::BorderImageWithBorderStyleNone);
+
     // FIXME: border-image is broken with full page zooming when tiling has to happen, since the tiling function
     // doesn't have any understanding of the zoom that is in effect on the tile.
     LayoutRect rectWithOutsets = rect;
     rectWithOutsets.expand(style.imageOutsets(ninePieceImage));
     IntRect borderImageRect = pixelSnappedIntRect(rectWithOutsets);
 
     IntSize imageSize = m_layoutObject.calculateImageIntrinsicDimensions(styleImage, borderImageRect.size(),
         LayoutBoxModelObject::DoNotScaleByEffectiveZoom);
 
     // If both values are 'auto' then the intrinsic width and/or height of the image should be used, if any.
@@ skipping 25 matching lines @@
                     drawInfo.tileRule.vertical, op);
             }
         }
     }
 
     graphicsContext->setImageInterpolationQuality(previousInterpolationQuality);
     return true;
 }
 
 } // namespace blink