Closed Bug 406903 Opened 12 years ago Closed 12 years ago

[FIX]"ASSERTION: Must be start token" with </input>

Categories

(Core :: HTML: Parser, defect)

x86
Windows XP
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla1.9beta3

People

(Reporter: martijn.martijn, Assigned: bzbarsky)

References

Details

(Keywords: assertion, regression, testcase)

Attachments

(2 files)

Attached file testcase
See testcase which triggers this assertion in current trunk build.
This assertion was added in bug 390565.
###!!! ASSERTION: Must be start token: 'eHTMLTokenTypes(aToken->GetTokenType())
== eToken_start', file c:/mozilla-build/mozilla/parser/htmlparser/src/CNavDTD.cp
p, line 494
Summary: ASSERTION: Must be start token with </input> → "ASSERTION: Must be start token" with </input>
Attached patch Silly fixSplinter Review
Assignee: nobody → bzbarsky
Status: NEW → ASSIGNED
Attachment #291601 - Flags: superreview?(mrbkap)
Attachment #291601 - Flags: review?(mrbkap)
Summary: "ASSERTION: Must be start token" with </input> → [FIX]"ASSERTION: Must be start token" with </input>
Target Milestone: --- → mozilla1.9 M11
Attachment #291601 - Flags: superreview?(mrbkap)
Attachment #291601 - Flags: superreview+
Attachment #291601 - Flags: review?(mrbkap)
Attachment #291601 - Flags: review+
Comment on attachment 291601 [details] [diff] [review]
Silly fix

Simple fix to pass a function only data it expects to see.
Attachment #291601 - Flags: approval1.9?
Attachment #291601 - Flags: approval1.9? → approval1.9+
Fixed.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.