Summary: | Optimize Element::getAttributeNS() by replacing String with AtomicString | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Kentaro Hara <haraken> | ||||||||
Component: | DOM | Assignee: | Kentaro Hara <haraken> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | ap, cmarcelo, rniwa, webkit.review.bot | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 90246 | ||||||||||
Attachments: |
|
Description
Kentaro Hara
2012-06-29 05:37:11 PDT
Created attachment 150156 [details]
Performance test
Created attachment 150158 [details]
Updated performance test
Created attachment 150159 [details]
Patch
Comment on attachment 150159 [details]
Patch
Based on further discussion in #webkit, my current understanding is that ap is ok with these patches. I'm going to r+ them, but please give ap a chance to respond before landing.
Comment on attachment 150159 [details] Patch Clearing flags on attachment: 150159 Committed r123944: <http://trac.webkit.org/changeset/123944> All reviewed patches have been landed. Closing bug. |