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

chrome/common/extensions/extension.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/common/extensions/extension.h"
 
 #include <algorithm>
 
 #include "base/base64.h"
 #include "base/basictypes.h"
@@ skipping 2739 matching lines @@
   // Don't show context menu for component extensions. We might want to show
   // options for component extension button but now there is no component
   // extension with options. All other menu items like uninstall have
   // no sense for component extensions.
   return location() != Extension::COMPONENT;
 }
 
 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 you're at it? We only really
care about blocking extensions.

[jochen (gone - plz use gerrit), 2011/12/12 20:42:59]

If I do that, I can't merge anymore - M16 doesn't have webRequestBlocking

[Matt Perry, 2011/12/12 21:08:40]

Oh, is it still possible to merge to M16? If so, carry on.

 }
 
 bool Extension::CanSpecifyAPIPermission(
     const ExtensionAPIPermission* permission,
     std::string* error) const {
   if (permission->is_component_only()) {
     if (!CanSpecifyComponentOnlyPermission()) {
       *error = ExtensionErrorUtils::FormatErrorMessage(
           errors::kPermissionNotAllowed, permission->name());
       return false;
@@ skipping 210 matching lines @@
     already_disabled(false),
     extension(extension) {}
 
 UpdatedExtensionPermissionsInfo::UpdatedExtensionPermissionsInfo(
     const Extension* extension,
     const ExtensionPermissionSet* permissions,
     Reason reason)
     : reason(reason),
       extension(extension),
       permissions(permissions) {}