Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(34)

Unified Diff: chrome/renderer/extensions/webstore_bindings.cc

Issue 12632004: Revert 186643 - Caused a 10% regression on SunSpider benchmark (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 7 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/renderer/extensions/webstore_bindings.cc
===================================================================
--- chrome/renderer/extensions/webstore_bindings.cc (revision 186747)
+++ chrome/renderer/extensions/webstore_bindings.cc (working copy)
@@ -53,7 +53,7 @@
WebstoreBindings::WebstoreBindings(Dispatcher* dispatcher,
ChromeV8Context* context)
- : ChromeV8Extension(dispatcher, context->v8_context()),
+ : ChromeV8Extension(dispatcher),
ChromeV8ExtensionHandler(context) {
RouteFunction("Install",
base::Bind(&WebstoreBindings::Install, base::Unretained(this)));
@@ -61,7 +61,7 @@
v8::Handle<v8::Value> WebstoreBindings::Install(
const v8::Arguments& args) {
- WebFrame* frame = WebFrame::frameForContext(v8_context());
+ WebFrame* frame = WebFrame::frameForCurrentContext();
if (!frame || !frame->view())
return v8::Undefined();
« no previous file with comments | « chrome/renderer/extensions/web_request_custom_bindings.cc ('k') | chrome/renderer/resources/extensions/apitest.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698