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

Unified Diff: chrome/browser/extensions/extension_host.cc

Issue 164039: Add module-level permissions to extensions. (Closed)
Patch Set: final nits Created 11 years, 4 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/browser/extensions/extension_host.cc
diff --git a/chrome/browser/extensions/extension_host.cc b/chrome/browser/extensions/extension_host.cc
index ee81c39a36c6f6662200f36e4126d795f99ff4c8..898aeaa01cf20e72911031ed9fe10e72e1214be7 100644
--- a/chrome/browser/extensions/extension_host.cc
+++ b/chrome/browser/extensions/extension_host.cc
@@ -386,4 +386,10 @@ Browser* ExtensionHost::GetBrowser() {
}
void ExtensionHost::RenderViewCreated(RenderViewHost* render_view_host) {
+ // TODO(mpcomplete): This is duplicated in DidNavigate, which means that
+ // we'll create 2 EFDs for the first navigation. We should try to find a
+ // better way to unify them.
+ // See http://code.google.com/p/chromium/issues/detail?id=18240
+ extension_function_dispatcher_.reset(
+ new ExtensionFunctionDispatcher(render_view_host, this, url_));
}
« no previous file with comments | « chrome/browser/extensions/extension_function_dispatcher.cc ('k') | chrome/browser/extensions/extensions_service_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698