Bug 39338

Summary: Add an HTML parsing benchmark
Product: WebKit Reporter: Adam Barth <abarth>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, eric, jamesr, mjs, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Attachments:
Description Flags
patch
none
A more awesome benchmark abarth: review+

Adam Barth
Reported 2010-05-18 20:23:35 PDT
Add an HTML parsing benchmark
Attachments
patch (deleted)
2010-05-18 20:27 PDT, Adam Barth 		no flags
DetailsFormatted DiffDiff
A more awesome benchmark (4.84 MB, patch)
2010-05-18 23:41 PDT, Eric Seidel (no email) 		abarth: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Adam Barth
Comment 1 2010-05-18 20:27:34 PDT
Created attachment 56445 [details] patch
WebKit Review Bot
Comment 2 2010-05-18 20:31:52 PDT
Attachment 56445 [details] did not pass style-queue: Failed to run "['WebKitTools/Scripts/check-webkit-style', '--no-squash']" exit_code: 1 WebCore/benchmarks/parser/resources/html5.html:75992: Line contains tab character. [whitespace/tab] [5] Total errors found: 1 in 3 files If any of these errors are false positives, please file a bug against check-webkit-style.
Eric Seidel (no email)
Comment 3 2010-05-18 23:41:32 PDT
Created attachment 56465 [details] A more awesome benchmark I used this to prove that bug 39318 was a regression and then fix it. :) Yay!
WebKit Review Bot
Comment 4 2010-05-18 23:46:27 PDT
Attachment 56465 [details] did not pass style-queue: Failed to run "['WebKitTools/Scripts/check-webkit-style', '--no-squash']" exit_code: 1 WebCore/benchmarks/parser/resources/html5.html:75992: Line contains tab character. [whitespace/tab] [5] Total errors found: 1 in 3 files If any of these errors are false positives, please file a bug against check-webkit-style.
Adam Barth
Comment 5 2010-05-18 23:49:58 PDT
Comment on attachment 56465 [details] A more awesome benchmark WebCore/benchmarks/parser/html-parser.html:6 + document.getElementById("log").innerText += text + "\n"; Boo
Adam Barth
Comment 6 2010-05-18 23:50:38 PDT
Comment on attachment 56465 [details] A more awesome benchmark This patch is breaking bugzilla.
WebKit Commit Bot
Comment 7 2010-05-19 01:22:21 PDT
Committed r59752: <http://trac.webkit.org/changeset/59752>
Note You need to log in before you can comment on or make changes to this bug.