Platform/HTML5 parser tasks: Difference between revisions

From MozillaWiki
Jump to navigation Jump to search
Line 9: Line 9:


* Fix "[HTML5]" bugs marked [https://bugzilla.mozilla.org/buglist.cgi?priority=P2;short_desc=%5bHTML5%5d;field0-0-0=short_desc;query_format=advanced;bug_status=NEW;bug_status=ASSIGNED;short_desc_type=allwordssubstr;type0-0-0=notsubstring;value0-0-0=%5bPatch%5d P2]. (hsivonen)
* Fix "[HTML5]" bugs marked [https://bugzilla.mozilla.org/buglist.cgi?priority=P2;short_desc=%5bHTML5%5d;field0-0-0=short_desc;query_format=advanced;bug_status=NEW;bug_status=ASSIGNED;short_desc_type=allwordssubstr;type0-0-0=notsubstring;value0-0-0=%5bPatch%5d P2]. (hsivonen)
** [https://bugzilla.mozilla.org/show_bug.cgi?id=482913 482913] Make sure document.write() order for nested scripts is right
***Waiting for Hixie to respond to feedback about a test case looking broken.
***''Time estimate'': Hixie's test cases for this can be run in less than a day. If something fails, working out what failed, why it failed and if we want to change code, tests or the spec will take multiple email round-trips through the WHATWG list spanning at least a week.
** [https://bugzilla.mozilla.org/show_bug.cgi?id=483209 483209] The HTML5 parser needs limits on internal buffer growth
** [https://bugzilla.mozilla.org/show_bug.cgi?id=483209 483209] The HTML5 parser needs limits on internal buffer growth
***''Time estimate'': Less than a day.
***''Time estimate'': Less than a day.
** [https://bugzilla.mozilla.org/show_bug.cgi?id=552908 552908] Implement spec changes to SVG </a> and </font> handling  
**Bugs waiting on Hixie or W3C WGs
***Waiting for Hixie to resolve bugs discovered in the spec text while implementing this.
*** [https://bugzilla.mozilla.org/show_bug.cgi?id=482913 482913] Make sure document.write() order for nested scripts is right
** [https://bugzilla.mozilla.org/show_bug.cgi?id=552938 552938] Implement the SVG load event in text/html
****Waiting for Hixie to respond to feedback about a test case looking broken.
***''Time estimate'': Less than a day, but waiting for the SVG WG to decide whether to remove this event instead.
****''Time estimate'': Hixie's test cases for this can be run in less than a day. If something fails, working out what failed, why it failed and if we want to change code, tests or the spec will take multiple email round-trips through the WHATWG list spanning at least a week.
** [https://bugzilla.mozilla.org/show_bug.cgi?id=557197 557197] &#13; in element content expands to CR and shows up as a non-collapsing space in layout  
*** [https://bugzilla.mozilla.org/show_bug.cgi?id=552908 552908] Implement spec changes to SVG </a> and </font> handling  
***''Time estimate'': Less than a day if hacked around in the parser. Unknown if fixed more properly in layout.
****Waiting for Hixie to resolve bugs discovered in the spec text while implementing this.
*** [https://bugzilla.mozilla.org/show_bug.cgi?id=552938 552938] Implement the SVG load event in text/html
****''Time estimate'': Less than a day, but waiting for the SVG WG to decide whether to remove this event instead.
*** [https://bugzilla.mozilla.org/show_bug.cgi?id=557197 557197] &amp;#13; in element content expands to CR and shows up as a non-collapsing space in layout  
****Preliminary layout patch exists. Waiting for the CSS WG to approve the solution or propose another solution.
*Security review.
*Security review.
** Talked to dveditz about how to do the security review for the HTML5 parser, and it's not clear what the best format for the review would be. But we did decide that it's not something we need to do before turning the parser on by default, but we need to do it before we ship it. --jst
** Talked to dveditz about how to do the security review for the HTML5 parser, and it's not clear what the best format for the review would be. But we did decide that it's not something we need to do before turning the parser on by default, but we need to do it before we ship it. --jst

Revision as of 13:26, 11 May 2010

This page outlines what needs to be done to ship a release with the HTML5 parser enabled by default.

High-priority fallout from turning the HTML5 parser on

  • 563536 Thread-safety assertion when loading charset aliases from a property file
    • Fix exists. Being fine-tuned on the tryserver for Ts effects.

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

  • Fix "[HTML5]" bugs marked P2. (hsivonen)
    • 483209 The HTML5 parser needs limits on internal buffer growth
      • Time estimate: Less than a day.
    • Bugs waiting on Hixie or W3C WGs
      • 482913 Make sure document.write() order for nested scripts is right
        • Waiting for Hixie to respond to feedback about a test case looking broken.
        • Time estimate: Hixie's test cases for this can be run in less than a day. If something fails, working out what failed, why it failed and if we want to change code, tests or the spec will take multiple email round-trips through the WHATWG list spanning at least a week.
      • 552908 Implement spec changes to SVG </a> and handling
        • Waiting for Hixie to resolve bugs discovered in the spec text while implementing this.
      • 552938 Implement the SVG load event in text/html
        • Time estimate: Less than a day, but waiting for the SVG WG to decide whether to remove this event instead.
      • 557197 &#13; in element content expands to CR and shows up as a non-collapsing space in layout
        • Preliminary layout patch exists. Waiting for the CSS WG to approve the solution or propose another solution.
  • Security review.
    • Talked to dveditz about how to do the security review for the HTML5 parser, and it's not clear what the best format for the review would be. But we did decide that it's not something we need to do before turning the parser on by default, but we need to do it before we ship it. --jst
  • 544994Create a "slow tp" test suite for exercising speculative parsing. (bnewman)
    • What if this suite reveals something bad?

Suspended items that may re-enter the list of release blockers

  • Suspended bugs that were previously P2
    • 482917 Make HTML5 parsing not regress stream data availability to extensions
      • There's now a patch but no way to test it.
      • Suspended, because there are no known extensions that use the API in question and are compatible with Firefox 3.6.
      • Time estimate: Unknown. (Need to find an extension that actually uses this. The alternative is writing an extension just for testing.)
    • 540480 Hootsuite.com dashboard not loading with HTML5 parser turned on
      • Moved over to evangelism.
    • 537948 The procedure to fix a <a> around a element is acting weird.
      • Spec issue. No indication so far on whether this is a problem for real-world sites. The HTML5 behavior seems well enough aligned with the IE8 behavior.