Bug 90242

Summary: [V8] Replace v8::Integer::New() with v8Integer() in custom bindings
Product: WebKit Reporter: Kentaro Hara <haraken>
Component: WebCore JavaScriptAssignee: Kentaro Hara <haraken>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, japhet, jochen, webkit.review.bot, yurys
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 84074    
Attachments:
Description Flags
Patch none

Kentaro Hara
Reported 2012-06-28 20:36:33 PDT
v8Integer() is a fast wrapper of v8::Integer::New(). We can replace v8::Integer::New() with v8Integer() in custom bindings.
Attachments
Patch (21.70 KB, patch)
2012-06-28 20:38 PDT, Kentaro Hara 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Kentaro Hara
Comment 1 2012-06-28 20:38:38 PDT
Created attachment 150074 [details] Patch
Yury Semikhatsky
Comment 2 2012-06-29 00:56:59 PDT
(In reply to comment #0) > v8Integer() is a fast wrapper of v8::Integer::New(). We can replace v8::Integer::New() with v8Integer() in custom bindings. Can you post performance measurement results here?
WebKit Review Bot
Comment 3 2012-06-29 02:50:46 PDT
Comment on attachment 150074 [details] Patch Clearing flags on attachment: 150074 Committed r121541: <http://trac.webkit.org/changeset/121541>
WebKit Review Bot
Comment 4 2012-06-29 02:50:51 PDT
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.