[Webkit-unassigned] [Bug 56027] Assert that editing does not ignore position's anchorNode if position is an offset in anchor

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Mar 9 10:50:25 PST 2011


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





--- Comment #3 from Ryosuke Niwa <rniwa at webkit.org>  2011-03-09 10:50:25 PST ---
(From update of attachment 85192)
View in context: https://bugs.webkit.org/attachment.cgi?id=85192&action=review

>> Source/WebCore/dom/Position.cpp:97
>> +    ASSERT(!m_anchorNode || !editingIgnoresContent(m_anchorNode.get()));
> 
> When is this ever called with a null anchorNode?

Numerous places.  I'm not sure if I can get rid of m_anchorNode in the final patch because it requires fixing lots of code.

>> Source/WebCore/dom/Position.cpp:103
>> +    ASSERT(!editingIgnoresContent(m_anchorNode.get()));
> 
> Why validate m_anchorNode here as well?  Do you mean "node" isntead?

Oops, that's what I meant.

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



More information about the webkit-unassigned mailing list