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

Unified Diff: chrome/common/extensions/extension.cc

Issue 8922014: Don't delay network requests for webNavigation, and re-enable the permission warning (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years 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
« no previous file with comments | « no previous file | chrome/common/extensions/extension_permission_set.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/common/extensions/extension.cc
diff --git a/chrome/common/extensions/extension.cc b/chrome/common/extensions/extension.cc
index 56f942134edb62bf21450d9b42c7f19a78857ba0..5a67fdf5fbc136177e64ace97331e8efb340778c 100644
--- a/chrome/common/extensions/extension.cc
+++ b/chrome/common/extensions/extension.cc
@@ -2757,8 +2757,7 @@ bool Extension::ShowConfigureContextMenus() const {
bool Extension::ImplicitlyDelaysNetworkStartup() const {
// Network requests should be deferred until any extensions that might want
// to observe or modify them are loaded.
- return HasAPIPermission(ExtensionAPIPermission::kWebNavigation) ||
- HasAPIPermission(ExtensionAPIPermission::kWebRequest);
+ return HasAPIPermission(ExtensionAPIPermission::kWebRequest);
Matt Perry 2011/12/12 20:29:55 Can you change this to kWebRequestBlocking while y
jochen (gone - plz use gerrit) 2011/12/12 20:42:59 If I do that, I can't merge anymore - M16 doesn't
Matt Perry 2011/12/12 21:08:40 Oh, is it still possible to merge to M16? If so, c
}
bool Extension::CanSpecifyAPIPermission(
« no previous file with comments | « no previous file | chrome/common/extensions/extension_permission_set.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698