RESOLVED FIXED90242
[V8] Replace v8::Integer::New() with v8Integer() in custom bindings 
https://bugs.webkit.org/show_bug.cgi?id=90242
Summary [V8] Replace v8::Integer::New() with v8Integer() in custom bindings
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.