Bug 52098 - [Meta] Need to cleanup Position, VisiblePosition, TextIterator, and visible_units
Summary: [Meta] Need to cleanup Position, VisiblePosition, TextIterator, and visible_u...
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Enhancement
Assignee: Nobody
URL:
Keywords:
Depends on: 52099 55354 55650 56161 63040 56027
Blocks:
  Show dependency treegraph
 
Reported: 2011-01-07 18:13 PST by Ryosuke Niwa
Modified: 2017-07-18 08:27 PDT (History)
10 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ryosuke Niwa 2011-01-07 18:13:03 PST
Right now, Position, VisiblePosition, and TextIterator each implements a logic to walk DOM tree visually.  Each one provides slightly different functionality and each has its own quirks. Because we have lot of inconsistencies and interdependencies, it has been a burden on people working on editing code.

We should cleanup these 3 classes in addition to functions provided in visible_units.cpp for Position/VisiblePosition.