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

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

Issue 113085: Split V8Proxy::retrieveActiveFrame() into two methods. (Closed)
Patch Set: Darin feedback Created 11 years, 7 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/extension_process_bindings.cc
diff --git a/chrome/renderer/extensions/extension_process_bindings.cc b/chrome/renderer/extensions/extension_process_bindings.cc
index 1a3614fd1c0ddc3bbf057f9295219cb04e5ceb8d..032d4570b4b2a3d2ead7a9480c798a85ef687206 100644
--- a/chrome/renderer/extensions/extension_process_bindings.cc
+++ b/chrome/renderer/extensions/extension_process_bindings.cc
@@ -69,8 +69,10 @@ class ExtensionImpl : public v8::Extension {
}
static v8::Handle<v8::Value> StartRequest(const v8::Arguments& args) {
- WebFrame* webframe = WebFrame::RetrieveActiveFrame();
- RenderView* renderview = GetActiveRenderView();
+ // Get the current RenderView so that we can send a routed IPC message from
+ // the correct source.
+ WebFrame* webframe = WebFrame::RetrieveFrameForCurrentContext();
+ RenderView* renderview = GetRenderViewForCurrentContext();
if (!webframe || !renderview)
return v8::Undefined();
« no previous file with comments | « chrome/renderer/extensions/bindings_utils.cc ('k') | chrome/renderer/extensions/renderer_extension_bindings.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698