Platform/HTML5 parser tasks: Difference between revisions
< Platform
Jump to navigation
Jump to search
(→Things that need to happen before turning the HTML5 parser on by default: Note that hsivonen needs to document stuff) |
|||
| Line 7: | Line 7: | ||
*Land 13 or so reviewed patches in hsivonen's queue (trivial once the tree is green). (hsivonen) | *Land 13 or so reviewed patches in hsivonen's queue (trivial once the tree is green). (hsivonen) | ||
*Get reviews on pending [https://bugzilla.mozilla.org/buglist.cgi?emailassigned_to1=1;query_format=advanced;short_desc=%5bHTML5%5d;field0-0-0=setters.login_name;bug_status=ASSIGNED;short_desc_type=allwordssubstr;email1=hsivonen%40iki.fi;type0-0-0=equals;value0-0-0=hsivonen%40iki.fi;emailtype1=exact patches]. (various people) | *Get reviews on pending [https://bugzilla.mozilla.org/buglist.cgi?emailassigned_to1=1;query_format=advanced;short_desc=%5bHTML5%5d;field0-0-0=setters.login_name;bug_status=ASSIGNED;short_desc_type=allwordssubstr;email1=hsivonen%40iki.fi;type0-0-0=equals;value0-0-0=hsivonen%40iki.fi;emailtype1=exact patches]. (various people) | ||
*Make tp4 not regress. (hsivonen) | *Make tp4 not [https://bugzilla.mozilla.org/show_bug.cgi?id=543458 regress]. (hsivonen) | ||
**A first go at the fix is in hand but it failed unit tests miserably. | |||
*Drive testsuite failures to zero. (hsivonen but help would be useful) | *Drive testsuite failures to zero. (hsivonen but help would be useful) | ||
*Either change form submission code to deal with IE-style isindex (hsivonen but help would be useful) or change the parser to use a Gecko-style isindex for now (hsivonen). | *Either change form submission code to deal with IE-style isindex (hsivonen but help would be useful) or change the parser to use a Gecko-style isindex for now (hsivonen). | ||
*Update the html5lib test suite snapshot used by mochitest. (jgriffin) | *[https://bugzilla.mozilla.org/show_bug.cgi?id=546635 Update] the html5lib test suite snapshot used by mochitest. (jgriffin) | ||
*Review the unreviewed parser core code. (sicking?) | *Review the unreviewed parser core code. (sicking?) | ||
**Some more code documentation on the Java side would be useful. (hsivonen) | **Some more code documentation on the Java side would be useful. (hsivonen) | ||
Revision as of 14:16, 25 February 2010
This page outlines what needs to happen to 1) enable the HTML5 parser by default on mozilla-central and 2) ship a release with the HTML5 parser enabled by default.
The HTML5 parser has been suitable for dogfooding for over a year and in the past couple of months there have been virtually no new site compat bugs. Thus, the parser is already alpha quality from a user POV but not from a zero test suite regressions POV.
Things that need to happen before turning the HTML5 parser on by default
- Land 13 or so reviewed patches in hsivonen's queue (trivial once the tree is green). (hsivonen)
- Get reviews on pending patches. (various people)
- Make tp4 not regress. (hsivonen)
- A first go at the fix is in hand but it failed unit tests miserably.
- Drive testsuite failures to zero. (hsivonen but help would be useful)
- Either change form submission code to deal with IE-style isindex (hsivonen but help would be useful) or change the parser to use a Gecko-style isindex for now (hsivonen).
- Update the html5lib test suite snapshot used by mochitest. (jgriffin)
- Review the unreviewed parser core code. (sicking?)
- Some more code documentation on the Java side would be useful. (hsivonen)
- Create a "slow tp" test suite for exercising speculative parsing. (bnewman) What if this suite reveals something bad?
Things that need to happen before shipping the HTML5 parser is a final Firefox release
- Fix "[HTML5]" bugs marked P2. (hsivonen)