560256 - [HTML5] Old document gets blown away when dynamically added script does document.write

Status: RESOLVED FIXED

(Core :: DOM: HTML Parser, defect, P2)

Description

[Martijn Wargers (dead)]

Follow-up from bug 553795.
See testcase, this passes in Mozilla 1.9.1 builds, but fails from 1.9.2 builds on (and also on trunk).
It passes in IE8.

It seems like a bug to me. If not, please resolve the bug as invalid and explain to me why it is invalid.

Comment 1

[Martijn Wargers (dead)]

Attachment: external script file

Comment 2

[Martijn Wargers (dead)]

Attachment: testcase

Comment 3

[Henri Sivonen (:hsivonen)]

This is invalid per spec, because script-inserted external scripts don't block the parser or defer EOF processing.

Comment 4

[Henri Sivonen (:hsivonen)]

Attachment: external script file that writes a non-empty string

Comment 5

[Henri Sivonen (:hsivonen)]

Attachment: Test case using non-empty strings

Comment 6

[Henri Sivonen (:hsivonen)]

OK. It seems that IE ignores the document.write() in this case.

Might be worth changing the spec. Now the interesting question is what criteria IE uses to decide whether to ignore the call or blow away the document.

Comment 7

[Henri Sivonen (:hsivonen)]

Filed as http://www.w3.org/Bugs/Public/show_bug.cgi?id=9767

If you know details about IE behavior, please comment on the w3.org bug.

Comment 8

[Henri Sivonen (:hsivonen)]

Attachment: Fix without tests or test adjustments

sicking, does this look worthy of writing and adjusting tests to match?

Comment 9

[Henri Sivonen (:hsivonen)]

Tests that need adjustment:

25085 ERROR TEST-UNEXPECTED-FAIL | /tests/content/base/test/test_bug518104.html | Div wasn't blown away. - 2 should equal 0

297 ERROR TEST-UNEXPECTED-FAIL | /tests/security/ssl/mixedcontent/test_bug329869.html | FAILURE: security broken after document write from unsecure script, expected broken got secure
298 ERROR TEST-UNEXPECTED-FAIL | /tests/security/ssl/mixedcontent/test_bug329869.html | FAILURE: for 'broken' expected  flags [0,1,0], security broken after document write from unsecure script
299 ERROR TEST-UNEXPECTED-FAIL | /tests/security/ssl/mixedcontent/test_bug329869.html | FAILURE: security still broken after navigation, expected broken got secure
300 ERROR TEST-UNEXPECTED-FAIL | /tests/security/ssl/mixedcontent/test_bug329869.html | FAILURE: for 'broken' expected  flags [0,1,0], security still broken after navigation

Comment 10

[Henri Sivonen (:hsivonen)]

Attachment: Fix with test adjustments

Comment 11

[Martijn Wargers (dead)]

I guess I'm seeing the same problem on:
http://www.voetbalprimeur.nl/site/nieuws/133916/Blatter_ziet_gevaar_in_technologische_hulpmiddelen.html

Comment 12

[Martijn Wargers (dead)]

I'm also seeing this when trying to watch an article on http://www.vastgoedjournaal.nl/

Comment 13

[Henri Sivonen (:hsivonen)]

http://hg.mozilla.org/mozilla-central/rev/85ce56b223a3

Comment 14

[Henri Sivonen (:hsivonen)]

(In reply to comment #11)
> I guess I'm seeing the same problem on:
> http://www.voetbalprimeur.nl/site/nieuws/133916/Blatter_ziet_gevaar_in_technologische_hulpmiddelen.html

(In reply to comment #12)
> I'm also seeing this when trying to watch an article on
> http://www.vastgoedjournaal.nl/

These sites work for me in the latest nightly.