Add extension permissions for new settings override API.

chrome/common/extensions/permissions/chrome_api_permissions.cc

Index: chrome/common/extensions/permissions/chrome_api_permissions.cc
diff --git a/chrome/common/extensions/permissions/chrome_api_permissions.cc b/chrome/common/extensions/permissions/chrome_api_permissions.cc
index 0fe74ed8f07c211fca79cd43e0fd6b3b748c0c15..65d88e070a5c6736a1f008b3b4b750a4421fa789 100644
--- a/chrome/common/extensions/permissions/chrome_api_permissions.cc
+++ b/chrome/common/extensions/permissions/chrome_api_permissions.cc
@@ -347,6 +347,23 @@ std::vector<APIPermissionInfo*> ChromeAPIPermissions::GetAllPermissions()
     { APIPermission::kFullscreen, "fullscreen" },
     { APIPermission::kAudio, "audio" },
     { APIPermission::kWebRtc, "webrtc" },
+
+    // Settings override permissions.
+    { APIPermission::kHomePage, "homePage",
+      APIPermissionInfo::kFlagCannotBeOptional |
+      APIPermissionInfo::kFlagInternal,

[gab, 2013/11/01 22:35:40]

Why internal? Is that how we make sure extensions with other permissions can't
add those?

If so, how does an extension actually specify these prefs? Is it the logic in
settings_overrides_handler.cc? Perhaps a comment here saying what takes care of
handling the internal logic makes sense?

[MAD, 2013/11/03 02:21:20]

This is what "internal" actually means, it means we insert these permissions
internally, as opposed to read them from the manifest.

+      IDS_EXTENSION_PROMPT_WARNING_HOME_PAGE_SETTING_OVERRIDE,
+      PermissionMessage::kHomePage },
+    { APIPermission::kSearchSettings, "searchSettings",
+      APIPermissionInfo::kFlagCannotBeOptional |
+      APIPermissionInfo::kFlagInternal,
+      IDS_EXTENSION_PROMPT_WARNING_SEARCH_SETTINGS_OVERRIDE,
+      PermissionMessage::kSearchSettings },
+    { APIPermission::kStartupPages, "startupPages",

[gab, 2013/11/01 22:35:40]

Other comments suggest this can only be 1 page; so should this be "startupPage"
instead?

[MAD, 2013/11/03 02:21:20]

Yes, but the manifest format calls this property startup_pages and have a list
type, so I prefer to keep it like this

+      APIPermissionInfo::kFlagCannotBeOptional |
+      APIPermissionInfo::kFlagInternal,
+      IDS_EXTENSION_PROMPT_WARNING_START_PAGE_SETTING_OVERRIDE,
+      PermissionMessage::kStartupPages },
   };
 
   std::vector<APIPermissionInfo*> permissions;