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

extensions/common/permissions/permission_message_util.h

 // Copyright 2014 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 EXTENSIONS_COMMON_PERMISSIONS_PERMISSION_MESSAGE_UTIL_H_
 #define EXTENSIONS_COMMON_PERMISSIONS_PERMISSION_MESSAGE_UTIL_H_
 
 #include <set>
 #include <string>
 
 namespace extensions {
 class PermissionMessage;
 class PermissionSet;
 class URLPatternSet;
 }
 
 namespace permission_message_util {
 
 // Creates the corresponding permission message for a list of hosts.
-// The messages change depending on what hosts are present.
+// The messages change depending on how many hosts are present, and whether
+// |read_only| is true.
+extensions::PermissionMessage CreateFromHostList(
+    const std::set<std::string>& hosts,
+    bool read_only);

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

ideally this would be an enum not a bool

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

Done.

 extensions::PermissionMessage CreateFromHostList(
     const std::set<std::string>& hosts);

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

could you remove this override? this only has 2 callers, one of which is
automation, so really you only need to update 1 caller.

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

Done.

 
 std::set<std::string> GetDistinctHosts(
     const extensions::URLPatternSet& host_patterns,
     bool include_rcd,
     bool exclude_file_scheme);
 
 }  // namespace permission_message_util
 
 #endif  // EXTENSIONS_COMMON_PERMISSIONS_PERMISSION_MESSAGE_UTIL_H_