Platform/HTML5 parser tasks: Difference between revisions

From MozillaWiki
Jump to navigation Jump to search
Line 5: Line 5:
== Things that need to be done before turning the HTML5 parser on by default ==
== Things that need to be done before turning the HTML5 parser on by default ==


*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 hsivonen's 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)
*[https://bugzilla.mozilla.org/show_bug.cgi?id=536097 536097]  spinning in nsHtml5TreeBuilder::startTag()?
*[https://bugzilla.mozilla.org/show_bug.cgi?id=536097 536097]  spinning in nsHtml5TreeBuilder::startTag()? (hsivonen)
**This will either be simple or so rare that it's not an alpha blocker after all.
**This will either be simple or so rare that it's not an alpha blocker after all.
*Drive testsuite failures to zero. (hsivonen but help would be useful)
**Need to reproduce this with debugger attached.
**Editor-related
*The following are waiting for landings by others
***[https://bugzilla.mozilla.org/show_bug.cgi?id=541078 541078]  contentEditable reload: docshell/test/navigation/test_bug430624.html fails
**[https://bugzilla.mozilla.org/show_bug.cgi?id=534458 534458]  test failure on css3.info selectors test with :checked{}
****Maybe fix in hand?
***Waiting on [https://bugzilla.mozilla.org/show_bug.cgi?id=502937 lazy frame construction] from Timothy Nikkel. (Seems reviewed and waiting landing.)
**XUL tests without any obvious HTML connection
**[https://bugzilla.mozilla.org/show_bug.cgi?id=543726 543726]  dom/tests/mochitest/ajax/offline/test_foreign.html fails
***[https://bugzilla.mozilla.org/show_bug.cgi?id=547688 547688]  toolkit/content/tests/widgets/test_radio.xul fails
***honzab has a fix. Waiting for review.
****No clue what this is about.
****Requested help from the author of the test case.
***[https://bugzilla.mozilla.org/show_bug.cgi?id=547934 547934] toolkit/content/tests/widgets/test_tooltip.xul times out
****No clue what this is about.
****Requested help from the author of the test case.
****Mac only
**Random?
***[https://bugzilla.mozilla.org/show_bug.cgi?id=546381 546381]  browser_bug537449.js fails
****Looks timing-sensitive. (Works locally on Mac and 64-Linux but fails on 32-bit Linux on the tryserver.)
****Requested help from the author of the test case.
**The following are waiting for landings by others
***[https://bugzilla.mozilla.org/show_bug.cgi?id=534458 534458]  test failure on css3.info selectors test with :checked{}
****Waiting on [https://bugzilla.mozilla.org/show_bug.cgi?id=502937 lazy frame construction] from Timothy Nikkel. (Seems reviewed and waiting landing.)
***[https://bugzilla.mozilla.org/show_bug.cgi?id=543726 543726]  dom/tests/mochitest/ajax/offline/test_foreign.html fails
****honzab has a fix. Waiting for review.
*[https://bugzilla.mozilla.org/show_bug.cgi?id=546635 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)

Revision as of 11:58, 9 March 2010

This page outlines what needs to be done 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 be done before turning the HTML5 parser on by default

  • Get reviews on hsivonen's pending patches. (various people)
  • 536097 spinning in nsHtml5TreeBuilder::startTag()? (hsivonen)
    • This will either be simple or so rare that it's not an alpha blocker after all.
    • Need to reproduce this with debugger attached.
  • The following are waiting for landings by others
    • 534458 test failure on css3.info selectors test with :checked{}
    • 543726 dom/tests/mochitest/ajax/offline/test_foreign.html fails
      • honzab has a fix. Waiting for review.
  • Update the html5lib test suite snapshot used by mochitest. (jgriffin)
  • Review the unreviewed parser core code. (sicking)
  • Create a "slow tp" test suite for exercising speculative parsing. (bnewman) What if this suite reveals something bad?
  • Security review.

Things that need to happen before shipping the HTML5 parser in a final Firefox release

  • Fix "[HTML5]" bugs marked P2. (hsivonen)