Don't show host permission warnings for platform apps.

chrome/common/extensions/permissions/permission_set.cc

Index: chrome/common/extensions/permissions/permission_set.cc
diff --git a/chrome/common/extensions/permissions/permission_set.cc b/chrome/common/extensions/permissions/permission_set.cc
index 5b51616c1e88b03e6795467af0a3966ae488d046..ea964db5bacf713b172e14727cfc984ad03163c9 100644
--- a/chrome/common/extensions/permissions/permission_set.cc
+++ b/chrome/common/extensions/permissions/permission_set.cc
@@ -264,8 +264,8 @@ std::set<std::string>
   return GetDistinctHosts(effective_hosts_, true, true);
 }
 
-PermissionMessages
-    PermissionSet::GetPermissionMessages() const {
+PermissionMessages PermissionSet::GetPermissionMessages(
+    const Extension* extension) const {
   PermissionMessages messages;
 
   if (HasEffectiveFullAccess()) {
@@ -275,14 +275,16 @@ PermissionMessages
     return messages;
   }
 
-  if (HasEffectiveAccessToAllHosts()) {
-    messages.push_back(PermissionMessage(
-        PermissionMessage::kHostsAll,
-        l10n_util::GetStringUTF16(IDS_EXTENSION_PROMPT_WARNING_ALL_HOSTS)));
-  } else {
-    std::set<std::string> hosts = GetDistinctHostsForDisplay();
-    if (!hosts.empty())
-      messages.push_back(PermissionMessage::CreateFromHostList(hosts));
+  if (!extension->is_platform_app()) {

[Aaron Boodman, 2012/08/01 23:13:05]

Consider passing in only the type enum since that is all you are using.

[Mihai Parparita -not on Chrome, 2012/08/02 00:33:44]

Done. Since this means #include-ing extension.h into permission_set.h, I had to
remove the permission_set.h #include in extension.h, which meant adding it a
bunch of places, that were relying on it appearing transitively. There's a TODO
about moving Extension::Type into its own header file, I'll look into doing
that.

+    if (HasEffectiveAccessToAllHosts()) {
+      messages.push_back(PermissionMessage(
+          PermissionMessage::kHostsAll,
+          l10n_util::GetStringUTF16(IDS_EXTENSION_PROMPT_WARNING_ALL_HOSTS)));
+    } else {
+      std::set<std::string> hosts = GetDistinctHostsForDisplay();
+      if (!hosts.empty())
+        messages.push_back(PermissionMessage::CreateFromHostList(hosts));
+    }
   }
 
   std::set<PermissionMessage> simple_msgs =
@@ -292,9 +294,10 @@ PermissionMessages
   return messages;
 }
 
-std::vector<string16> PermissionSet::GetWarningMessages() const {
+std::vector<string16> PermissionSet::GetWarningMessages(
+    const Extension* extension) const {
   std::vector<string16> messages;
-  PermissionMessages permissions = GetPermissionMessages();
+  PermissionMessages permissions = GetPermissionMessages(extension);
 
   bool audio_capture = false;
   bool video_capture = false;