Closed Bug 524447 Opened 10 years ago Closed 10 years ago

[HTML5] Tokenize on document.close()

Categories

(Core :: DOM: HTML Parser, defect)

Product:
Core
Component:
DOM: HTML Parser
Type:
defect
Priority:
Not set

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: hsivonen, Assigned: hsivonen)

References

(
URL
)

Details

Attachments

(1 file)

Tokenize on EOF
3.00 KB, patch
mozilla+ben
: review+
Details | Diff | Splinter Review 
IE, WebKit and Opera tokenize on EOF. For interop, Gecko should too.

Implement
http://html5.org/tools/web-apps-tracker?from=4240&to=4241
Attached patch Tokenize on EOF — — Splinter Review
Assignee: nobody → hsivonen
Status: NEW → ASSIGNED
Attachment #408378 - Flags: review?(bnewman)
Blocks: 518595
Attachment #408378 - Flags: review?(bnewman) → review+
Hmm. Weird. This patch doesn't exhibit the same behavior as WebKit and IE.

On the surface, it looks like Gecko blocks the parser more eagerly than the other engines.
So it seems this patch is reasonable in itself, but in order to get compat with WebKit/IE/Presto, the script loader or sink needs to behave differently. Gecko unblocks the parser after both the test case document.close() and the harness document.close() in this case:
http://software.hixie.ch/utilities/js/live-dom-viewer/saved/276
http://hg.mozilla.org/mozilla-central/rev/c61a7e1570c0
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.