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

[bugzilla-daemon at webkit.org]

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





------- 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.