Bug 15122 - REGRESSION: Assertion failure in avoidIntersectionWithNode (!node || node->parentNode()) running editing/deleting/5408255.html
Summary: REGRESSION: Assertion failure in avoidIntersectionWithNode (!node || node->pa...
Alias: None
Product: WebKit
Classification: Unclassified
Component: HTML Editing (show other bugs)
Version: 523.x (Safari 3)
Hardware: Macintosh OS X 10.4
: P2 Normal
Assignee: Darin Adler
URL: http://build.webkit.org/results/post-...
Keywords: InRadar, LayoutTestFailure, Regression
Depends on:
Reported: 2007-08-31 02:05 PDT by mitz
Modified: 2007-08-31 09:20 PDT (History)
2 users (show)

See Also:

patch that fixes assertion (2.18 KB, patch)
2007-08-31 09:17 PDT, Darin Adler
andersca: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description mitz 2007-08-31 02:05:48 PDT
editing/deleting/5408255.html fails in debug builds with
ASSERTION FAILED: !node || node->parentNode()
(/WebKit/WebCore/editing/htmlediting.cpp:915 WebCore::Selection WebCore::avoidIntersectionWithNode(const WebCore::Selection&, WebCore::Node*))

<http://build.webkit.org/post-commit-leaks-intel-mac-os-x/builds/1074> is the first build to show the failure. The assertion itself was changed in <http://trac.webkit.org/projects/webkit/changeset/25311>.
Comment 1 Mark Rowe (bdash) 2007-08-31 02:07:35 PDT
Comment 2 Darin Adler 2007-08-31 09:12:12 PDT
This is happening because the delete button controller no longer destroys its container element when not in use. So it can be hanging around with no parent.

We're OK as long as we don't actually try to work with the container element, so I just moved the assertion.
Comment 3 Darin Adler 2007-08-31 09:17:24 PDT
Created attachment 16169 [details]
patch that fixes assertion
Comment 4 Anders Carlsson 2007-08-31 09:18:28 PDT
Comment on attachment 16169 [details]
patch that fixes assertion

Comment 5 Darin Adler 2007-08-31 09:20:07 PDT
Committed revision 25330.