Bug 57073

Summary: NamedNodeMaps created from Atomic Tokens use 4 MB on Membuster
Product: WebKit Reporter: Stephanie Lewis <slewis>
Component: DOMAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: slewis
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
patch ggaren: review+

Stephanie Lewis
Reported 2011-03-24 19:45:51 PDT
The AtomicHTMLToken function reserved space for ten tokens in the attribute map which was rarely filled. Rework the function to reserve space for the number of attributes in the list. Seems slightly faster on the parser benchmarks. Saves 1.5 MB on Membuster.
Attachments
patch (3.82 KB, patch)
2011-03-24 19:51 PDT, Stephanie Lewis 		ggaren: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Stephanie Lewis
Comment 1 2011-03-24 19:51:34 PDT
Created attachment 86872 [details] patch
Geoffrey Garen
Comment 2 2011-03-24 20:18:21 PDT
Comment on attachment 86872 [details] patch r=me
Stephanie Lewis
Comment 3 2011-03-24 23:01:10 PDT
committed http://trac.webkit.org/projects/webkit/changeset/81937
Note You need to log in before you can comment on or make changes to this bug.