Bug 126049

Summary: Remove a significant source of memory allocations during CSS parsing
Product: WebKit Reporter: Ryosuke Niwa <rniwa>
Component: CSSAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: barraclough, benjamin, cdumez, joepeck, kling, koivisto
Priority: P2 Keywords: BlinkMergeCandidate
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   

Description Ryosuke Niwa 2013-12-19 21:47:01 PST 
Consider merging https://chromium.googlesource.com/chromium/blink/+/3d1744e53a3540be9bdb385deadec60469e6019c
or come up with a similar improvement

This is about +2% on Parser/css-parser-yui.html locally.

Previously, we were allocating and freeing small vector buffers (most commonly
24 or 72 byte allocations on 64-bit), only to immediately throw them away.

Note that uncheckedAppend has an ASSERT that will cleanly catch any future
problems with sizing.
Comment 1 Joseph Pecoraro 2016-04-28 17:11:20 PDT 
Still worth taking? Seems like a pretty small change, but might be out of date now.
Comment 2 Chris Dumez 2016-04-29 09:20:35 PDT 
I don't believe this is still needed after:
https://bugs.webkit.org/show_bug.cgi?id=140599

*** This bug has been marked as a duplicate of bug 140599 ***