Bug 24587 - PositionIterator should be re-written using Position
Summary: PositionIterator should be re-written using Position
Status: NEW
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-03-13 14:17 PDT by Eric Seidel (no email)
Modified: 2012-06-01 00:52 PDT (History)
4 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 2009-03-13 14:17:42 PDT 
PositionIterator should be re-written using Position

PositionIterator is a large chunk of mostly copied code from Position.  I think with a little work, PositionIterator could be re-written using Position objects.

For example:
bool PositionIterator::isCandidate() const
bool Position::isCandidate() const

void PositionIterator::increment()
Position Position::next(PositionMoveType moveType) const

are basically just copies of one another.