Bug 104084

Summary: [V8] Replace String::New("symbol") with String::NewSymbol("symbol")
Product: WebKit Reporter: Kentaro Hara <haraken>
Component: WebCore JavaScriptAssignee: Kentaro Hara <haraken>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, japhet, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Kentaro Hara
Reported 2012-12-04 21:21:21 PST
V8 can look up symbols faster than strings. This is a final patch for the replacement.
Attachments
Patch (23.94 KB, patch)
2012-12-04 21:22 PST, Kentaro Hara 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Kentaro Hara
Comment 1 2012-12-04 21:22:29 PST
Created attachment 177664 [details] Patch
WebKit Review Bot
Comment 2 2012-12-04 22:21:51 PST
Comment on attachment 177664 [details] Patch Clearing flags on attachment: 177664 Committed r136641: <http://trac.webkit.org/changeset/136641>
WebKit Review Bot
Comment 3 2012-12-04 22:21:55 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.