[Webkit-unassigned] [Bug 60910] might need to bundle the functions related to visual word break as a class (VisualWordBreaker)

[bugzilla-daemon at webkit.org]

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


Van Lam <vanlam at google.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #101518|0                           |1
        is obsolete|                            |
 Attachment #101541|                            |review?
               Flag|                            |




--- Comment #12 from Van Lam <vanlam at google.com>  2011-07-20 18:33:13 PST ---
Created an attachment (id=101541)
 --> (https://bugs.webkit.org/attachment.cgi?id=101541&action=review)
Revised fix

Made requested changes; now:

1) leftWordPosition and rightWordPosition are called through non-member wrapper functions of the same names
2) the class definition is in visible_units.cpp; the header file's dependency on VisiblePosition is removed

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