Bug 165869

Summary: [CSS Parser] Enable CSS Deferred Parsing
Product: WebKit Reporter: Dave Hyatt <hyatt>
Component: CSSAssignee: Dave Hyatt <hyatt>
Status: REOPENED    
Severity: Normal CC: barraclough, commit-queue
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Dave Hyatt
Reported 2016-12-14 14:21:35 PST
This bug is tracking turning on CSS deferred parsing by default. I'm going to post some performance numbers to make it clear what a win it is.
Attachments
Patch (4.23 KB, patch)
2016-12-14 15:25 PST, Dave Hyatt 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Dave Hyatt
Comment 1 2016-12-14 14:53:27 PST
Some #s. All are done on reloads of the page to ensure all stylesheets reparse. 10 loads done of each site with deferred parsing on and off and average taken. 40% reduction on apple.com 25% reduction on twitter.com (logged in with a feed) 25% reduction on facebook.com (logged in, my account) 30% reduction on cnn.com The gain is anywhere from 5-15ms for each page in the examples above. Parsing doesn't take all that long on most pages, but even so, this is a huge improvement.
Dave Hyatt
Comment 2 2016-12-14 15:25:18 PST
Created attachment 297136 [details] Patch
Sam Weinig
Comment 3 2016-12-14 16:11:26 PST
Comment on attachment 297136 [details] Patch Nice nice!
WebKit Commit Bot
Comment 4 2016-12-15 07:46:49 PST
Comment on attachment 297136 [details] Patch Clearing flags on attachment: 297136 Committed r209862: <http://trac.webkit.org/changeset/209862>
WebKit Commit Bot
Comment 5 2016-12-15 07:46:53 PST
All reviewed patches have been landed. Closing bug.
Gavin Barraclough
Comment 6 2016-12-19 16:06:50 PST
Reverted in r209997 due to performance regressions.
Note You need to log in before you can comment on or make changes to this bug.