Don't count host permission changes for v2 apps as a permission increase.

chrome/common/extensions/permissions/permission_set.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_COMMON_EXTENSIONS_PERMISSIONS_PERMISSION_SET_H_
 #define CHROME_COMMON_EXTENSIONS_PERMISSIONS_PERMISSION_SET_H_
 
 #include <map>
 #include <set>
 #include <string>
@@ skipping 108 matching lines @@
 
   // Returns true if this permission set includes effective access to |url|.
   bool HasEffectiveAccessToURL(const GURL& url) const;
 
   // Returns ture if this permission set effectively represents full access
   // (e.g. native code).
   bool HasEffectiveFullAccess() const;
 
   // Returns true if |permissions| has a greater privilege level than this
   // permission set (e.g., this permission set has less permissions).
-  bool HasLessPrivilegesThan(const PermissionSet* permissions) const;
+  // Whether certain permissions are considered varies by extension type.
+  bool HasLessPrivilegesThan(const PermissionSet* permissions,
+                             Manifest::Type extension_type) const;
 
   const APIPermissionSet& apis() const { return apis_; }
 
   const URLPatternSet& effective_hosts() const { return effective_hosts_; }
 
   const URLPatternSet& explicit_hosts() const { return explicit_hosts_; }
 
   const URLPatternSet& scriptable_hosts() const { return scriptable_hosts_; }
 
  private:
@@ skipping 23 matching lines @@
   void InitImplicitPermissions();
 
   // Initializes the effective host permission based on the data in this set.
   void InitEffectiveHosts();
 
   // Gets the permission messages for the API permissions.
   std::set<PermissionMessage> GetSimplePermissionMessages() const;
 
   // Returns true if |permissions| has an elevated API privilege level than
   // this set.
-  bool HasLessAPIPrivilegesThan(
-      const PermissionSet* permissions) const;
+  bool HasLessAPIPrivilegesThan(const PermissionSet* permissions) const;
 
   // Returns true if |permissions| has more host permissions compared to this
   // set.
-  bool HasLessHostPrivilegesThan(
-      const PermissionSet* permissions) const;
+  bool HasLessHostPrivilegesThan(const PermissionSet* permissions,
+                                 Manifest::Type extension_type) const;
 
   // Gets a list of the distinct hosts for displaying to the user.
   // NOTE: do not use this for comparing permissions, since this disgards some
   // information.
   std::set<std::string> GetDistinctHostsForDisplay() const;
 
   // The api list is used when deciding if an extension can access certain
   // extension APIs and features.
   APIPermissionSet apis_;
 
   // The list of hosts that can be accessed directly from the extension.
   // TODO(jstritar): Rename to "hosts_"?
   URLPatternSet explicit_hosts_;
 
   // The list of hosts that can be scripted by content scripts.
   // TODO(jstritar): Rename to "user_script_hosts_"?
   URLPatternSet scriptable_hosts_;
 
   // The list of hosts this effectively grants access to.
   URLPatternSet effective_hosts_;
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_COMMON_EXTENSIONS_PERMISSIONS_PERMISSION_SET_H_