Bug 16511

Summary: Speed up ClassNodeList and NamedNodeList by using the caching mechanism employed by ChildNodeLists
Product: WebKit Reporter: Sam Weinig <sam>
Component: DOMAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
patch
sam: review-
second try mjs: review+

Sam Weinig
Reported 2007-12-18 20:10:37 PST
This turned out to be quite easy and a big win.
Attachments
patch (13.23 KB, patch)
2007-12-18 20:12 PST, Sam Weinig 		sam: review-
DetailsFormatted DiffDiff
second try (14.11 KB, patch)
2007-12-18 23:06 PST, Sam Weinig 		mjs: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Sam Weinig
Comment 1 2007-12-18 20:12:41 PST
Created attachment 17979 [details] patch
Sam Weinig
Comment 2 2007-12-18 20:21:14 PST
Ignore the stuff in DRT, that was not intended to be there.
Sam Weinig
Comment 3 2007-12-18 20:29:16 PST
Comment on attachment 17979 [details] patch Scratch that, there seems to be an invalidation error I missed. Will return shortly.
Sam Weinig
Comment 4 2007-12-18 23:06:49 PST
Created attachment 17982 [details] second try This time with less test failures!
Maciej Stachowiak
Comment 5 2007-12-19 01:06:25 PST
Comment on attachment 17982 [details] second try r=me
Sam Weinig
Comment 6 2007-12-19 12:11:39 PST
Landed in r28868.
Note You need to log in before you can comment on or make changes to this bug.