From: eric@webkit.org Date: Wed, 2 Feb 2011 23:32:39 +0000 (+0000) Subject: 2011-01-27 Eric Seidel X-Git-Url: https://git.webkit.org/?p=WebKit.git;a=commitdiff_plain;h=a9551841333633ad8f9aff6e3425d5272e6192b7;hp=8e2e5f5690e7d83d9e403db1ba1c8a8e8a3279ba 2011-01-27 Eric Seidel Reviewed by Darin Adler. HTML5 TreeBuilder regressed a Peacekeeper DOM test by 40% https://bugs.webkit.org/show_bug.cgi?id=48719 It's unclear exactly what the Peacekeeper benchmark is testing, because I haven't found a way to run it myself. However, I constructed a benchmark which shows at least one possible slow point. The HTML5 spec talks about creating a new document for every time we use the fragment parsing algorithm. Document() it turns out, it a huge bloated mess, and the constructor and destructor do a huge amount of work. * benchmarks/parser/tiny-innerHTML.html: Added. git-svn-id: http://svn.webkit.org/repository/webkit/trunk@77426 268f45cc-cd09-0410-ab3c-d52691b4dbfc --- diff --git a/PerformanceTests/Parser/ChangeLog b/PerformanceTests/Parser/ChangeLog index d3bc5c6d04e6..48d5eaef759e 100644 --- a/PerformanceTests/Parser/ChangeLog +++ b/PerformanceTests/Parser/ChangeLog @@ -1,3 +1,20 @@ +2011-01-27 Eric Seidel + + Reviewed by Darin Adler. + + HTML5 TreeBuilder regressed a Peacekeeper DOM test by 40% + https://bugs.webkit.org/show_bug.cgi?id=48719 + + It's unclear exactly what the Peacekeeper benchmark is testing, + because I haven't found a way to run it myself. + + However, I constructed a benchmark which shows at least one possible slow point. + The HTML5 spec talks about creating a new document for every time we use + the fragment parsing algorithm. Document() it turns out, it a huge bloated + mess, and the constructor and destructor do a huge amount of work. + + * benchmarks/parser/tiny-innerHTML.html: Added. + 2011-01-29 Sheriff Bot Unreviewed, rolling out r77050. diff --git a/PerformanceTests/Parser/tiny-innerHTML.html b/PerformanceTests/Parser/tiny-innerHTML.html new file mode 100644 index 000000000000..b357cf1a128a --- /dev/null +++ b/PerformanceTests/Parser/tiny-innerHTML.html @@ -0,0 +1,16 @@ + + +

+ + +