Commit 2bfa5488 authored by jochen@chromium.org's avatar jochen@chromium.org

Don't delay network requests for webNavigation, and re-enable the permission warning

BUG=107254,107255
TEST=as described in the bug

Review URL: http://codereview.chromium.org/8922014

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@114100 0039d316-1c4b-4281-b951-d872f2087c98
parent c3456bbc
...@@ -2757,8 +2757,7 @@ bool Extension::ShowConfigureContextMenus() const { ...@@ -2757,8 +2757,7 @@ bool Extension::ShowConfigureContextMenus() const {
bool Extension::ImplicitlyDelaysNetworkStartup() const { bool Extension::ImplicitlyDelaysNetworkStartup() const {
// Network requests should be deferred until any extensions that might want // Network requests should be deferred until any extensions that might want
// to observe or modify them are loaded. // to observe or modify them are loaded.
return HasAPIPermission(ExtensionAPIPermission::kWebNavigation) || return HasAPIPermission(ExtensionAPIPermission::kWebRequest);
HasAPIPermission(ExtensionAPIPermission::kWebRequest);
} }
bool Extension::CanSpecifyAPIPermission( bool Extension::CanSpecifyAPIPermission(
......
...@@ -271,8 +271,9 @@ void ExtensionAPIPermission::RegisterAllPermissions( ...@@ -271,8 +271,9 @@ void ExtensionAPIPermission::RegisterAllPermissions(
kTtsEngine, "ttsEngine", IDS_EXTENSION_PROMPT_WARNING_TTS_ENGINE, kTtsEngine, "ttsEngine", IDS_EXTENSION_PROMPT_WARNING_TTS_ENGINE,
ExtensionPermissionMessage::kTtsEngine, kFlagNone, kTypeDefault); ExtensionPermissionMessage::kTtsEngine, kFlagNone, kTypeDefault);
info->RegisterPermission( info->RegisterPermission(
kWebNavigation, "webNavigation", 0, kWebNavigation, "webNavigation",
ExtensionPermissionMessage::kNone, kFlagSupportsOptional, IDS_EXTENSION_PROMPT_WARNING_TABS,
ExtensionPermissionMessage::kTabs, kFlagSupportsOptional,
kTypeDefault - kTypePlatformApp); kTypeDefault - kTypePlatformApp);
info->RegisterPermission( info->RegisterPermission(
kWebRequest, "webRequest", 0, kWebRequest, "webRequest", 0,
......
...@@ -648,10 +648,9 @@ TEST(ExtensionPermissionSetTest, PermissionMessages) { ...@@ -648,10 +648,9 @@ TEST(ExtensionPermissionSetTest, PermissionMessages) {
// permissions. // permissions.
skip.insert(ExtensionAPIPermission::kCookie); skip.insert(ExtensionAPIPermission::kCookie);
// The proxy, webNavigation, and webRequest permissions are warned as part of // The proxy, and webRequest permissions are warned as part of host
// host permission checks. // permission checks.
skip.insert(ExtensionAPIPermission::kProxy); skip.insert(ExtensionAPIPermission::kProxy);
skip.insert(ExtensionAPIPermission::kWebNavigation);
skip.insert(ExtensionAPIPermission::kWebRequest); skip.insert(ExtensionAPIPermission::kWebRequest);
skip.insert(ExtensionAPIPermission::kWebRequestBlocking); skip.insert(ExtensionAPIPermission::kWebRequestBlocking);
......
...@@ -4,5 +4,5 @@ ...@@ -4,5 +4,5 @@
"manifest_version": 2, "manifest_version": 2,
"name": "My startup delay extension 2", "name": "My startup delay extension 2",
"background_page": "background.html", "background_page": "background.html",
"permissions": ["tabs", "webNavigation"] "permissions": ["tabs", "webRequest"]
} }
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment