Show read-only messages for non-interact non-desktop Automation extensions

extensions/common/permissions/permission_message_util.cc

Index: extensions/common/permissions/permission_message_util.cc
diff --git a/extensions/common/permissions/permission_message_util.cc b/extensions/common/permissions/permission_message_util.cc
index 070eabd2f0285cf3eb0efc8b50f8fa23f07acfa4..825447c08458cd861f65d3bc1685709c4a199d6b 100644
--- a/extensions/common/permissions/permission_message_util.cc
+++ b/extensions/common/permissions/permission_message_util.cc
@@ -38,8 +38,14 @@ bool RcdBetterThan(const std::string& a, const std::string& b) {
 namespace permission_message_util {
 
 PermissionMessage CreateFromHostList(const std::set<std::string>& hosts) {
+  return CreateFromHostList(hosts, false);
+}
+
+PermissionMessage CreateFromHostList(const std::set<std::string>& hosts,
+                                     bool read_only) {
   typedef std::pair<PermissionMessage::ID, int> MsgPair;
-  const MsgPair kMessagesList[] = {
+  static const int kNumMessages = 4;

[not at google - send to devlin, 2014/07/25 18:17:42]

we went to great lengths to avoid hard-coding a number here.

could you:
- remove it,
- do COMPILE_ASSERT(arraysize(kReadWriteMessagesList) ==
arraysize(kReadOnlyMessagesList)
- (the part I'm unsure about) make messages_list a pointer not an array
- either continue to use arraysize(kMessagesList) or assign it to a value like
messages_list_size.

[aboxhall, 2014/07/25 19:06:31]

Done.
Done.
It's not an array, it's a reference.
Done (but it's assigned to a static const value).

+  static const MsgPair kMessagesList[kNumMessages] = {
       std::make_pair(PermissionMessage::kHosts1,
                      IDS_EXTENSION_PROMPT_WARNING_1_HOST),
       std::make_pair(PermissionMessage::kHosts2,
@@ -48,8 +54,19 @@ PermissionMessage CreateFromHostList(const std::set<std::string>& hosts) {
                      IDS_EXTENSION_PROMPT_WARNING_3_HOSTS),
       std::make_pair(PermissionMessage::kHosts4OrMore,
                      IDS_EXTENSION_PROMPT_WARNING_HOSTS_LIST)};
-
-  int host_msg_id = hosts.size() < arraysize(kMessagesList)
+  static const MsgPair kReadOnlyMessagesList[kNumMessages] = {
+      std::make_pair(PermissionMessage::kHosts1ReadOnly,
+                     IDS_EXTENSION_PROMPT_WARNING_1_HOST_READ_ONLY),
+      std::make_pair(PermissionMessage::kHosts2ReadOnly,
+                     IDS_EXTENSION_PROMPT_WARNING_2_HOSTS_READ_ONLY),
+      std::make_pair(PermissionMessage::kHosts3ReadOnly,
+                     IDS_EXTENSION_PROMPT_WARNING_3_HOSTS_READ_ONLY),
+      std::make_pair(PermissionMessage::kHosts4OrMoreReadOnly,
+                     IDS_EXTENSION_PROMPT_WARNING_HOSTS_LIST_READ_ONLY)};
+  const MsgPair (&messages_list)[kNumMessages] =
+      read_only ? kReadOnlyMessagesList : kMessagesList;
+
+  int host_msg_id = hosts.size() < kNumMessages

[meacer, 2014/07/25 18:13:10]

nit: This style seems to be more common:

bool ?
true :
false;
 

[aboxhall, 2014/07/25 19:06:31]

Funny: this is how clang-format formats it. I'm going to leave as-is for that
reason.

                         ? IDS_EXTENSION_PROMPT_WARNING_HOST_AND_SUBDOMAIN
                         : IDS_EXTENSION_PROMPT_WARNING_HOST_AND_SUBDOMAIN_LIST;
   std::vector<base::string16> host_list;
@@ -65,11 +82,11 @@ PermissionMessage CreateFromHostList(const std::set<std::string>& hosts) {
   }
   DCHECK(host_list.size());
 
-  if (host_list.size() < arraysize(kMessagesList)) {
+  if (host_list.size() < kNumMessages) {
     return PermissionMessage(
-        kMessagesList[host_list.size() - 1].first,
+        messages_list[host_list.size() - 1].first,
         l10n_util::GetStringFUTF16(
-            kMessagesList[host_list.size() - 1].second, host_list, NULL));
+            messages_list[host_list.size() - 1].second, host_list, NULL));
   }
 
   base::string16 details;
@@ -80,9 +97,9 @@ PermissionMessage CreateFromHostList(const std::set<std::string>& hosts) {
         IDS_EXTENSION_PROMPT_WARNING_HOST_LIST_ENTRY, host_list[i]);
   }
   return PermissionMessage(
-      kMessagesList[arraysize(kMessagesList) - 1].first,
+      messages_list[arraysize(kMessagesList) - 1].first,
       l10n_util::GetStringUTF16(
-          kMessagesList[arraysize(kMessagesList) - 1].second),
+          messages_list[arraysize(kMessagesList) - 1].second),
       details);
 }