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

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

Issue 7888064: Only deliver extension messages to contexts that care. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Nail down a bit further. Only poke relevant RenderView in the case of chrome.tabs.connect(). Created 9 years, 3 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/bindings_utils.cc
diff --git a/chrome/renderer/extensions/bindings_utils.cc b/chrome/renderer/extensions/bindings_utils.cc
index 3cad5ea3714fe8625dc3706d669cceac8a4e62f2..066d1307cce8eb86e4075cec01e39725f7e845fb 100644
--- a/chrome/renderer/extensions/bindings_utils.cc
+++ b/chrome/renderer/extensions/bindings_utils.cc
@@ -38,6 +38,18 @@ ContextInfo::ContextInfo(v8::Persistent<v8::Context> context,
ContextInfo::~ContextInfo() {}
+WebFrame* ContextInfo::GetWebFrame() const {
+ return WebFrame::frameForContext(context);
+}
+
+RenderView* ContextInfo::GetRenderView() const {
+ WebFrame* frame = GetWebFrame();
+ if (!frame || !frame->view())
+ return NULL;
+
+ return RenderView::FromWebView(frame->view());
+}
+
ContextList& GetContexts() {
return g_singleton_data.Get().contexts;
}

Powered by Google App Engine
This is Rietveld 408576698