[Webkit-unassigned] [Bug 61344] --webkit-visual-word does not work in multi-line

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=61344





--- Comment #6 from Ryosuke Niwa <rniwa at webkit.org>  2011-06-01 19:54:15 PST ---
(From update of attachment 95693)
View in context: https://bugs.webkit.org/attachment.cgi?id=95693&action=review

>>> Source/WebCore/editing/visible_units.cpp:1469
>>> +static const RootInlineBox* previousRootInlineBox(const InlineBox* box)
>> 
>> This seems like a very expensive function.  Why do we need to do this?
> 
> RootInlineBox->prevLineBox() and nextLineBox() only catches the multiple lines caused by wrapping or by <br> in <div>abc def<br>hij opq</div>.
> For the <br> in <div>abc def <div><br></div>hij opq</div> (or the newline entered by user in contenteditable div), we need to find the previousRootInlineBox in order to go from "hij opq" to "abc def".

Can't we just walk on render tree?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.