483644 - XML Parsing Error when loading this page (works in Firefox 3.0.7)

Status: VERIFIED FIXED

(Core :: DOM: HTML Parser, defect)

Description

[Marcia Knous [:marcia]]

Reported from a Hendrix user:

STR:
1. Load the page listed in the URL

Actual: XML Parsing error
Expected: Page would load normally as it does in FF 3.0.7 with no errors

Comment 1

[Nochum Sossonko [:Natch]]

dupe of bug 478889?

Comment 2

[Ria Klaassen (not reading all bugmail)]

Probably

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2]]

(In reply to comment #1)
> dupe of bug 478889?

Lets ask Boris, but at least in the code of the given web page I cannot see a deferred attribute.

Comment 4

[Boris Zbarsky [:bzbarsky]]

The regression range is the same as for bug 478889.  That said, I don't see any deferred scripts here either (including via setting the relevant breakpoints in nsScriptLoader).

That said, the last patch in bug 478889 does seem to fix it.  Let's leave it dependent for now, dup if the patch that lands fixes it.

Comment 5

[Nochum Sossonko [:Natch]]

This works for me now with latest m-c (i.e. with the patch for bug 478889).

Comment 6

[Boris Zbarsky [:bzbarsky]]

Great.

Would be really nice to have a testcase, of course... :)

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+2]]

(In reply to comment #6)
> Would be really nice to have a testcase, of course... :)

Martijn, would you mind (if you have time)?

Verified fixed with builds on OSX and Windows: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2a1pre) Gecko/20090318 Minefield/3.6a1pre ID:20090318052249

Comment 8

[Martijn Wargers (dead)]

Attachment: testcase

Sorry for the delay, it turns out this is causing a parser error before this bug was fixed.

Comment 9

[Boris Zbarsky [:bzbarsky]]

Martijn, do you think you can make this into a reftest or mochitest?

Comment 10

[Martijn Wargers (dead)]

Yes, will do. Sorry, I still have a backlog of other things to do.

Comment 12

[Johnathan Nightingale [:johnath]]

Pretty clear that this was fixed on branch, marking it. Obviously it would still be good to get the mochi/reftest, but I don't think this needs to show up on the blocker report any more.

Comment 13

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Verified fixed on 1.9.1 with builds on OS X and Windows:

Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b5pre) Gecko/20090513 Shiretoko/3.5b5pre ID:20090513034609