Closed Bug 508867 Opened 15 years ago Closed 11 years ago

Update HTML5 mochitests once HTML5 parser is on by default

Categories

(Core :: DOM: HTML Parser, defect)

defect
Not set
normal

Tracking

()

RESOLVED INVALID

People

(Reporter: jgriffin, Unassigned)

References

Details

We have a number of mochitests that test things by enabling the HTML5 pref (html5.enable), and then running a test.  Once the HTML5 parser is enabled by default, these tests should be updated to remove this code.
Bugs on particular Mochitests, even when those particular tests constitute large swaths, go in a component relevant to the function being tested.
Component: Mochitest → HTML: Parser
Product: Testing → Core
QA Contact: mochitest → parser
Henri, since HTML5 is enabled by default, is it safe (on trunk) to remove the code from the html5 tree-construction tests that sets the html5.enable preference?
Oops. Sorry about my failure to respond earlier.

Earlier, the plan was to keep the option to go back to the old HTML parser as a contingency plan. However, now new work has been done that depends on the capabilities of the HTML5 parser.

smaug, Does your work on bug 462076 and 562013 make it infeasible to keep supporting html5.enable=false?

jst, What's your take on making test suites explicitly no longer support html5.enable=false?
obsolete
Assignee: jgriffin → nobody
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.