Summary: | Shrink ShadowRoot and TreeScope. | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Andreas Kling <kling> | ||||
Component: | DOM | Assignee: | Andreas Kling <kling> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | koivisto, ojan, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Andreas Kling
2012-11-27 15:44:46 PST
Created attachment 176351 [details]
Proposed patch
Comment on attachment 176351 [details]
Proposed patch
r=me
Comment on attachment 176351 [details]
Proposed patch
Mind adding COMPILE_ASSERTS for the sizes of TreeScope and ShadowRoot? Ideally you'd commit that first so that we could see the improvement from this patch.
(In reply to comment #3) > (From update of attachment 176351 [details]) > Mind adding COMPILE_ASSERTS for the sizes of TreeScope and ShadowRoot? Ideally you'd commit that first so that we could see the improvement from this patch. Sure, will do! Comment on attachment 176351 [details] Proposed patch Clearing flags on attachment: 176351 Committed r135939: <http://trac.webkit.org/changeset/135939> All reviewed patches have been landed. Closing bug. (In reply to comment #4) > (In reply to comment #3) > > (From update of attachment 176351 [details] [details]) > > Mind adding COMPILE_ASSERTS for the sizes of TreeScope and ShadowRoot? Ideally you'd commit that first so that we could see the improvement from this patch. > > Sure, will do! Aw crap, forgot about the CQ. Well, I'll slap some compile asserts on there anyway. Better late than never. Yeah, no worries. Better late than never indeed! |