Bug 3624 – REGRESSION repro crash: asserction faliure in dom_position.cpp:494 (dev build only)

• Component: DOM
• Status: VERIFIED FIXED

When I go to http://www.dhl.com/splash.html, choose Israel from the pop-up menu, and press Go.
ToT crashes immediately with the following message:

/Users/dan/Documents/Development/WebKit/WebCore/khtml/xml/dom_position.cpp:494: failed 
assertion `currentPos == start'
Abort

The crash doesn't happen with WebKit-412.6

Now it's OK again.

(In reply to comment #1)
> Now it's OK again.

Actually it's still broken.

[Joost de Valk (AlthA)]

Can't confirm this one, sorry. Works on ToT.

(In reply to comment #3)
> Can't confirm this one, sorry. Works on ToT.

Works here part of the time, too, but always fails if it's the first site I go to after launching Safari.

[Joost de Valk (AlthA)]

still can't confirm...

Created attachment 2692 [details]
testcase

I can trigger the crash by opening this file in Safari an clicking inside the
yellow rect, somewhere to the left of the red arrow.

OK, a clean build seems to have solved it for real. Sorry.

(In reply to comment #7)
> OK, a clean build seems to have solved it for real. Sorry.

Actually, that was a clean *deployment* build (where assert() does nothing?). A clean development build 
still crashes.

[Oliver Hunt]

I can reproduce this on ToT dev build -- may not be reproducible on deployment build as it's an assertion 
failure

[Mark Rowe (bdash)]

I can easily reproduce this with a development build of WebKit by clicking once inside the yellow rectangle 
to the left of the red triangle.  As previous comments note the assertion failure is not triggered with a 
deployment build.

[Joost de Valk (AlthA)]

well my bad, apologies for that one :).

Fixed (by removing asserts) by David Harrison (as fix for <rdar://problem/4119646> <rdar://problem/
4125949>).