Bug 73780

Summary: Convert JSClassRef to use HashMap<OwnPtr>
Product: WebKit Reporter: Darin Adler <darin>
Component: JavaScriptCoreAssignee: Darin Adler <darin>
Status: RESOLVED FIXED    
Severity: Normal CC: webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 73757    
Attachments:
Description Flags
Patch none

Darin Adler
Reported 2011-12-04 14:26:29 PST
Convert JSClassRef to use HashMap<OwnPtr>
Attachments
Patch (13.46 KB, patch)
2011-12-04 14:31 PST, Darin Adler 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Darin Adler
Comment 1 2011-12-04 14:31:04 PST
Created attachment 117808 [details] Patch
Andreas Kling
Comment 2 2011-12-04 18:49:08 PST
Comment on attachment 117808 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=117808&action=review > Source/JavaScriptCore/API/JSClassRef.cpp:82 > // Use a local variable here to sidestep an RVCT compiler bug. Side-stepping RVCT bugs is not an issue anymore AFAIK. Support for Symbian has been removed from WebKit trunk, and I believe it was the only user of RVCT. (This comment repeated for each occurrence of that workaround.)
WebKit Review Bot
Comment 3 2011-12-05 16:17:53 PST
Comment on attachment 117808 [details] Patch Clearing flags on attachment: 117808 Committed r102065: <http://trac.webkit.org/changeset/102065>
WebKit Review Bot
Comment 4 2011-12-05 16:17:57 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.