183784 - Crash when running DOM testsuite

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[Peter Van der Beken [:peterv]]

Go to http://www.bclary.com/dom-ts/do-dom-html-2-tests.html, click the
"Initialize jsUnit" button. A new page will open. Click the "Run" button. You'll
crash. Also see bug 171117 and attachment 108361 [details].
Bz has a patch

Comment 1

[Boris Zbarsky [:bzbarsky]]

Attachment: I hate document.write... I hate parser blocking. Back it all out.

Comment 2

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 108397 [details] [diff] [review]
I hate document.write... I hate parser blocking.  Back it all out.

This crash will happen on any page that document.writes multiple links to the
same URL....

Comment 3

[Asa Dotzler [:asa]]

Comment on attachment 108397 [details] [diff] [review]
I hate document.write... I hate parser blocking.  Back it all out.

please hold off on requesting approval until you've secured review and
super-review. thanks.

Comment 4

[Peter Van der Beken [:peterv]]

Comment on attachment 108397 [details] [diff] [review]
I hate document.write... I hate parser blocking.  Back it all out.

BAH!

Comment 5

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Comment on attachment 108397 [details] [diff] [review]
I hate document.write... I hate parser blocking.  Back it all out.

r=sicking

Comment 6

[Asa Dotzler [:asa]]

Comment on attachment 108397 [details] [diff] [review]
I hate document.write... I hate parser blocking.  Back it all out.

a=asa for checkin to 1.3a

Comment 7

[Boris Zbarsky [:bzbarsky]]

checked in

Comment 8

[Peter Van der Beken [:peterv]]

Attachment: testcase (part 1)

Comment 9

[Peter Van der Beken [:peterv]]

Attachment: testcase (part 2)

Comment 10

[Peter Van der Beken [:peterv]]

Attachment: testcase (part 3)

Comment 11

[Peter Van der Beken [:peterv]]

If you open attachment 18502 [details] [diff] [review], and click the button you'll hit the multiple
parser problem bz mentions.