Move the tts and ttsEngine APIs out of experimental, and give

chrome/common/extensions/extension_permission_set.cc

 // Copyright (c) 2011 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 "chrome/common/extensions/extension_permission_set.h"
 
 #include <algorithm>
 #include <string>
 
 #include "base/memory/singleton.h"
@@ skipping 281 matching lines @@
       ExtensionPermissionMessage::kNone);
   RegisterExtensionPermission(
       ExtensionAPIPermission::kManagement, "management",
       IDS_EXTENSION_PROMPT_WARNING_MANAGEMENT,
       ExtensionPermissionMessage::kManagement);
   RegisterExtensionPermission(
       ExtensionAPIPermission::kTab, "tabs",
       IDS_EXTENSION_PROMPT_WARNING_TABS,
       ExtensionPermissionMessage::kTabs);
   RegisterExtensionPermission(
+      ExtensionAPIPermission::kTts, "tts",
+      IDS_EXTENSION_PROMPT_WARNING_TTS,

[Erik does not do reviews, 2011/07/15 23:04:04]

Do we really need a TTS warning?  My take is that this falls into the
"annoyance" category like notifications and quota that we don't warn for.

[dmazzoni, 2011/07/20 05:58:04]

Done.

+      ExtensionPermissionMessage::kTts);
+  RegisterExtensionPermission(
+      ExtensionAPIPermission::kTtsEngine, "ttsEngine",
+      IDS_EXTENSION_PROMPT_WARNING_TTS_ENGINE,

[Erik does not do reviews, 2011/07/15 23:04:04]

This one seems fine though.

+      ExtensionPermissionMessage::kTtsEngine);
+  RegisterExtensionPermission(
       ExtensionAPIPermission::kWebSocketProxyPrivate,
       "webSocketProxyPrivate", 0,
       ExtensionPermissionMessage::kNone);
 
   // Private permissions
   RegisterPrivatePermission(
       ExtensionAPIPermission::kChromeosInfoPrivate, "chromeosInfoPrivate");
   RegisterPrivatePermission(
       ExtensionAPIPermission::kFileBrowserPrivate, "fileBrowserPrivate");
   RegisterPrivatePermission(
@@ skipping 463 matching lines @@
   std::set<std::string> new_hosts_set = GetDistinctHosts(new_list, false);
   std::set<std::string> old_hosts_set = GetDistinctHosts(old_list, false);
   std::set<std::string> new_hosts_only;
 
   std::set_difference(new_hosts_set.begin(), new_hosts_set.end(),
                       old_hosts_set.begin(), old_hosts_set.end(),
                       std::inserter(new_hosts_only, new_hosts_only.begin()));
 
   return !new_hosts_only.empty();
 }