Bug 69285 – Add a microbenchmark for a full-page render of the HTML5 spec

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Eric Seidel (no email)]

Add a microbenchmark for a full-page render of the HTML5 spec

[Eric Seidel (no email)]

Created attachment 109517 [details]
Patch

[Adam Barth]

Comment on attachment 109517 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=109517&action=review

> PerformanceTests/Parser/html-parser.html:11
>      var iframe = document.createElement("iframe");
> -    iframe.src = "about:blank";
>      iframe.style.display = "none";  // Prevent creation of the rendering tree, so we only test HTML parsing.
> -    iframe.sandbox = '';  // Prevents loading of external scripts which would otherwise pause the parser.
> +    iframe.sandbox = '';  // Prevent external script loads which could cause write() to return before completing the parse.

This landed separately, right?

[Adam Barth]

Comment on attachment 109517 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=109517&action=review

>> PerformanceTests/Parser/html-parser.html:11
>> +    iframe.sandbox = '';  // Prevent external script loads which could cause write() to return before completing the parse.
> 
> This landed separately, right?

Oh, i see.  You're just updating this.

[Antti Koivisto]

Must be the first time ever the single page HTML5 spec has been called a "microbenchmark" :)

[WebKit Review Bot]

Comment on attachment 109517 [details]
Patch

Clearing flags on attachment: 109517

Committed r96559: <http://trac.webkit.org/changeset/96559>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.