Resolve flickering the text selection when hitting culled inline.

Source/core/rendering/RenderInline.cpp

Index: Source/core/rendering/RenderInline.cpp
diff --git a/Source/core/rendering/RenderInline.cpp b/Source/core/rendering/RenderInline.cpp
index 9956e851279d96343b3233742bd5654fc45590f5..0b2b0f259306dba6f2a4971ae39ecfe1c416bfd2 100644
--- a/Source/core/rendering/RenderInline.cpp
+++ b/Source/core/rendering/RenderInline.cpp
@@ -835,7 +835,7 @@ PositionWithAffinity RenderInline::positionForPoint(const LayoutPoint& point)
 {
     // FIXME: Does not deal with relative positioned inlines (should it?)
     RenderBlock* cb = containingBlock();
-    if (firstLineBox()) {
+    if (firstLineBoxIncludingCulling()) {

[pdr., 2015/02/16 20:24:09]

There are 35 other callers of firstLineBox(). Is this the only one that needs to
be changed? How do we know which to use?

[Miyoung Shin(g), 2015/02/17 05:01:54]

We need only this to solve the problem that flickers on the text selection.
But I'm not sure if we need to change firstLineBox to
firstLineBoxIncludingCulling more for other cases.
IMHO, we should choose firstLineBoxIncludingCulling() if we want to calculate
the position such as offsetLeft() and generateLineBoxRects().
But we don't need to use it for the painting job.
For example,
I think we don't need to use firstLineBoxIncludingCulling in
RenderInline::invalidateDisplayItemClients,
because culled inline doesn't have own displaying area.

As you said, there are many other callers of firstLineBox(),
but I can see to branch out the code flow between the normal inline and culled
inline using alwaysCreateLineBoxes() in almost places calling firstLineBox().

For example.
RenderInline::linesBoundingBox() {
...
    if (!alwaysCreateLineBoxes()) {
	...
	}
..
}

If my comment is not clear, let me know.

         // This inline actually has a line box.  We must have clicked in the border/padding of one of these boxes.  We
         // should try to find a result by asking our containing block.
         return cb->positionForPoint(point);