Add a border radius property to inherit it from a parent in a fallback content

third_party/WebKit/Source/core/html/HTMLImageFallbackHelper.cpp

 // Copyright 2014 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 "core/html/HTMLImageFallbackHelper.h"
 
 #include "core/HTMLNames.h"
 #include "core/InputTypeNames.h"
 #include "core/dom/ElementRareData.h"
 #include "core/dom/Text.h"
@@ skipping 26 matching lines @@
     RefPtrWillBeRawPtr<HTMLDivElement> container = HTMLDivElement::create(element.document());
     root.appendChild(container);
     container->setAttribute(idAttr, AtomicString("alttext-container", AtomicString::ConstructFromLiteral));
     container->setInlineStyleProperty(CSSPropertyOverflow, CSSValueHidden);
     container->setInlineStyleProperty(CSSPropertyBorderWidth, 1, CSSPrimitiveValue::UnitType::Pixels);
     container->setInlineStyleProperty(CSSPropertyBorderStyle, CSSValueSolid);
     container->setInlineStyleProperty(CSSPropertyBorderColor, CSSValueSilver);
     container->setInlineStyleProperty(CSSPropertyDisplay, CSSValueInlineBlock);
     container->setInlineStyleProperty(CSSPropertyBoxSizing, CSSValueBorderBox);
     container->setInlineStyleProperty(CSSPropertyPadding, 1, CSSPrimitiveValue::UnitType::Pixels);
+    container->setInlineStyleProperty(CSSPropertyBorderRadius, AtomicString("inherit", AtomicString::ConstructFromLiteral));

[pdr., 2016/03/31 04:20:11]

I still don't see why this is needed. Which test fails without this change?

[Miyoung Shin(c), 2016/04/03 14:39:25]

border-hittest-with-image-fallback.html is failed without this change.

BTW, I checked how to draw the missed image having border-radius on FireFox,
they apply the rounded border to the missed image but we don't.
I think we need to draw the rounded border of the missed image and it works fine
with this path like them.

 
     RefPtrWillBeRawPtr<HTMLImageElement> brokenImage = HTMLImageElement::create(element.document());
     container->appendChild(brokenImage);
     brokenImage->setIsFallbackImage();
     brokenImage->setAttribute(idAttr, AtomicString("alttext-image", AtomicString::ConstructFromLiteral));
     brokenImage->setAttribute(widthAttr, AtomicString("16", AtomicString::ConstructFromLiteral));
     brokenImage->setAttribute(heightAttr, AtomicString("16", AtomicString::ConstructFromLiteral));
     brokenImage->setAttribute(alignAttr, AtomicString("left", AtomicString::ConstructFromLiteral));
     brokenImage->setInlineStyleProperty(CSSPropertyMargin, 0, CSSPrimitiveValue::UnitType::Pixels);
 
@@ skipping 48 matching lines @@
     // This preserves legacy behaviour originally defined when alt-text was managed by LayoutImage.
     if (noImageSourceSpecified(element))
         brokenImage->setInlineStyleProperty(CSSPropertyDisplay, CSSValueNone);
     else
         brokenImage->setInlineStyleProperty(CSSPropertyDisplay, CSSValueInline);
 
     return newStyle;
 }
 
 } // namespace blink