[Webkit-unassigned] [Bug 24760] Clean up Position.h

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Mar 23 11:35:23 PDT 2009


------- Comment #7 from darin at apple.com  2009-03-23 11:35 PDT -------
(In reply to comment #4)
> The main goal at the end of this rewrite, is to get rid of the [node, 0] can
> mean two different things depending on what editingIgnoresContent(node)
> returns.

I think it’s a great idea to create a way to represent before and after
positions without having to compute a node index. This does make a fundamental
change in Position, because Position now has capabilities that a plain old
container/index pair does not.

My biggest concern is that if you do this, you programmers will have a
difficult choice between using PassRefPtr<Node> or Node* with no reference
count churn, or Position with greater capabilities and a stronger model, but
mandatory reference count churn.

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

More information about the webkit-unassigned mailing list