WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
102008
[V8] We should be able to get V8PerContextData from a v8::Context more quickly
https://bugs.webkit.org/show_bug.cgi?id=102008
Summary
[V8] We should be able to get V8PerContextData from a v8::Context more quickly
Adam Barth
Reported
2012-11-12 16:37:47 PST
[V8] We should be able to get V8PerContextData from a v8::Context more quickly
Attachments
Patch
(4.59 KB, patch)
2012-11-12 16:39 PST
,
Adam Barth
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Adam Barth
Comment 1
2012-11-12 16:39:51 PST
Created
attachment 173760
[details]
Patch
WebKit Review Bot
Comment 2
2012-11-12 20:46:53 PST
Comment on
attachment 173760
[details]
Patch Clearing flags on attachment: 173760 Committed
r134351
: <
http://trac.webkit.org/changeset/134351
>
WebKit Review Bot
Comment 3
2012-11-12 20:46:57 PST
All reviewed patches have been landed. Closing bug.
Ojan Vafai
Comment 4
2012-11-13 10:25:24 PST
It's hard to be sure it was this patch, but it looks like this was a 18% improvement on dromaeo_jslibeventprototype/jslib_event_prototype_Prototype___fire:
http://build.chromium.org/f/chromium/perf/chromium-rel-win7-webkit/dromaeo_jslibeventprototype/report.html?graph=jslib_event_prototype_Prototype___fire&history=150&rev=167405
Ojan Vafai
Comment 5
2012-11-16 11:13:44 PST
8% improvement on
http://build.chromium.org/f/chromium/perf/linux-release-webkit-latest/dromaeo_jslibstylejquery/report.html?rev=167458&graph=jslib_style_jquery_jQuery____is__visible_&history=150
10% improvement on
http://build.chromium.org/f/chromium/perf/linux-release-webkit-latest/dromaeo_jslibstylejquery/report.html?rev=167602&graph=jslib_style_jquery_jQuery____show__&history=150
11% improvement on
http://build.chromium.org/f/chromium/perf/linux-release-webkit-latest/dromaeo_jslibstylejquery/report.html?rev=167628&graph=jslib_style_jquery_jQuery___css_color__x100&history=150
There's a bunch more, but collating them all is too time consuming. Nice patch!
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug