[Webkit-unassigned] [Bug 110408] New: Tune BackgroundHTMLParser's pendingTokenLimit based on a benchmark

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=110408

           Summary: Tune BackgroundHTMLParser's pendingTokenLimit based on
                    a benchmark
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebCore Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: tonyg at chromium.org
                CC: eric at webkit.org, abarth at webkit.org
            Blocks: 106127


I ran the threaded HTML parser in Telemetry against the top 25 sites on a Nexus 7 with a variety of pendingTokenLimits ranging from 250-6000 (current == 4000).

My observations:
- The threaded HTML parser improves by DOMContentLoaded 8-14% and total time spent parsing on main thread by 9-13%
- Varying pendingTokenLimit doesn't significantly change total time spent parsing on main thread, DOMContentLoaded or Load
- Tweaking pendingTokenLimit can improve maximum time spent parsing on main thread by another 40%

I recommend we change it from 4000 to 1000.

Results:
https://docs.google.com/a/chromium.org/spreadsheet/ccc?key=0AmVDuVhIZxCTdEstXy02NXZnNE5wb0d4NTlPNGQ2U1E#gid=1

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.