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

third_party/WebKit/Source/core/layout/line/InlineBox.cpp

Index: third_party/WebKit/Source/core/layout/line/InlineBox.cpp
diff --git a/third_party/WebKit/Source/core/layout/line/InlineBox.cpp b/third_party/WebKit/Source/core/layout/line/InlineBox.cpp
index d07c203204359f08a871b8f826515b7569cea595..3e3e6426c6330aa0963504e4db147a76f81b283b 100644
--- a/third_party/WebKit/Source/core/layout/line/InlineBox.cpp
+++ b/third_party/WebKit/Source/core/layout/line/InlineBox.cpp
@@ -226,6 +226,8 @@ bool InlineBox::nodeAtPoint(HitTestResult& result, const HitTestLocation& locati
     if (parent()->lineLayoutItem().hasFlippedBlocksWritingMode()) // Faster than calling containingBlock().
         childPoint = layoutObject().containingBlock()->flipForWritingModeForChild(&toLayoutBox(layoutObject()), childPoint);
 
+    bool inside = lineLayoutItem().hitTest(result, locationInContainer, childPoint);

[pdr., 2016/01/31 21:32:36]

The other nodeAtPoint functions follow a pattern where, for non-rect hit
testing, the HitTestResult is updated if and only if nodeAtPoint returns true.
For example, in LayoutBlock::nodeAtPoint, all of the codepaths that return true
will call updateHitTestResult (except for rect-based tests). In this patch,
you're updating the result (through lineLayoutItem().hitTest) but returning
false from InlineBox::nodeAtPoint... that doesn't seem right. Can you
investigate why a different pattern is used here vs the other nodeAtPoints?

It may also be useful to find what code is working when InlineBox::nodeAtPoint
returns false but the HitTestResult has been updated. I suspect that code is
wrong.

[Miyoung Shin(c), 2016/02/05 23:40:03]

Your right. I found out we have only this issue with missing image.
We create a new element for missing image, but don't inherit the style from the
original IMG.
If we apply the right style, I think we don't need this code.
Thanks for letting me look deep into this.

It's a National holiday until a next week.
After I come back, I will upload the new patchset.

+
     if (lineLayoutItem().style()->hasBorderRadius()) {
         LayoutRect borderRect = logicalFrameRect();
         borderRect.moveBy(accumulatedOffset);
@@ -233,8 +235,7 @@ bool InlineBox::nodeAtPoint(HitTestResult& result, const HitTestLocation& locati
         if (!locationInContainer.intersects(border))
             return false;
     }
-
-    return lineLayoutItem().hitTest(result, locationInContainer, childPoint);
+    return inside;
 }
 
 const RootInlineBox& InlineBox::root() const