Added PermissionIDSet to APIPermissions

extensions/common/permissions/settings_override_permission.cc

 // 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.
 
 #include "extensions/common/permissions/settings_override_permission.h"
 
 #include "base/strings/utf_string_conversions.h"
+#include "extensions/common/permissions/api_permission_set.h"
 #include "grit/extensions_strings.h"
 #include "ui/base/l10n/l10n_util.h"
 
 namespace extensions {
 
 SettingsOverrideAPIPermission::SettingsOverrideAPIPermission(
     const APIPermissionInfo* permission,
     const std::string& setting_value)
     : APIPermission(permission), setting_value_(setting_value) {}
 
 SettingsOverrideAPIPermission::~SettingsOverrideAPIPermission() {}
 
+PermissionIDSet SettingsOverrideAPIPermission::GetPermissions() const {
+  PermissionIDSet permissions;
+  permissions.insert(info()->id(), base::UTF8ToUTF16(setting_value_));
+  return permissions;
+}
+
 bool SettingsOverrideAPIPermission::HasMessages() const {
   return info()->message_id() > PermissionMessage::kNone;
 }
 
 PermissionMessages SettingsOverrideAPIPermission::GetMessages() const {
   DCHECK(HasMessages());
   int string_id = -1;
+  // Warning: when modifying this function, be sure to modify the correct rule
+  // in ChromePermissionMessageProvider.
   switch (id()) {
     case kHomepage: {
+      // TODO(sashab): Add a parameter rule in ChromePermissionMessageProvider:
+      // kHomepage -> IDS_EXTENSION_PROMPT_WARNING_HOME_PAGE_SETTING_OVERRIDE
       string_id = IDS_EXTENSION_PROMPT_WARNING_HOME_PAGE_SETTING_OVERRIDE;
       break;
     }
     case kStartupPages: {
+      // TODO(sashab): Add a parameter rule in ChromePermissionMessageProvider:
+      // kStartupPages ->
+      // IDS_EXTENSION_PROMPT_WARNING_START_PAGE_SETTING_OVERRIDE
       string_id = IDS_EXTENSION_PROMPT_WARNING_START_PAGE_SETTING_OVERRIDE;
       break;
     }
     case kSearchProvider: {
+      // TODO(sashab): Add a parameter rule in ChromePermissionMessageProvider:
+      // kSearchProvider ->
+      // IDS_EXTENSION_PROMPT_WARNING_SEARCH_SETTINGS_OVERRIDE
       string_id = IDS_EXTENSION_PROMPT_WARNING_SEARCH_SETTINGS_OVERRIDE;
       break;
     }
     default:
       NOTREACHED();
   }
   PermissionMessages result;
   result.push_back(
       PermissionMessage(info()->message_id(),
                         l10n_util::GetStringFUTF16(
@@ skipping 53 matching lines @@
 void SettingsOverrideAPIPermission::Write(IPC::Message* m) const {}
 
 bool SettingsOverrideAPIPermission::Read(const IPC::Message* m,
                                          PickleIterator* iter) {
   return true;
 }
 
 void SettingsOverrideAPIPermission::Log(std::string* log) const {}
 
 }  // namespace extensions