Closed Bug 547685 Opened 14 years ago Closed 14 years ago

[HTML5][Patch] parser/htmlparser/tests/mochitest/test_bug418464.html fails

Categories

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

x86
Linux
defect

Tracking

()

RESOLVED FIXED

People

(Reporter: hsivonen, Assigned: hsivonen)

Details

Attachments

(1 file)

140607 ERROR TEST-UNEXPECTED-FAIL | /tests/parser/htmlparser/tests/mochitest/test_bug418464.html | Should not have a form here - got [[object HTMLInputElement]], expected undefined
Assignee: nobody → hsivonen
Status: NEW → ASSIGNED
Attachment #428180 - Flags: review?(bzbarsky)
Summary: [HTML5] parser/htmlparser/tests/mochitest/test_bug418464.html fails → [HTML5][Patch] parser/htmlparser/tests/mochitest/test_bug418464.html fails
Comment on attachment 428180 [details] [diff] [review]
Weaken the test not to require the first script to fail to find the form

I would prefer that this become todo_is(form1 instanceof HTMLFormElement).

r=bzbarsky with that.
Attachment #428180 - Flags: review?(bzbarsky) → review+
(In reply to comment #2)
> (From update of attachment 428180 [details] [diff] [review])
> I would prefer that this become todo_is(form1 instanceof HTMLFormElement).

I can't make it a todo, since that would give an UNEXPECTED-PASS with the HTML5 parser. I could comment the line out and file a bug about uncommenting it in the future. Would your r hold for that?
Sounds good, yeah.
OK. The reminder bug is bug 547963.
Thanks. Pushed:
http://hg.mozilla.org/mozilla-central/rev/62c81fdaeba1
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: