Bug 102008 – [V8] We should be able to get V8PerContextData from a v8::Context more quickly

• Component: New Bugs
• Status: RESOLVED FIXED

[Adam Barth]

[V8] We should be able to get V8PerContextData from a v8::Context more quickly

[Adam Barth]

Created attachment 173760 [details]
Patch

[WebKit Review Bot]

Comment on attachment 173760 [details]
Patch

Clearing flags on attachment: 173760

Committed r134351: <http://trac.webkit.org/changeset/134351>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.

[Ojan Vafai]

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]

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!