Expose microphone content setting to extensions.

chrome/browser/extensions/api/content_settings/content_settings_helpers.cc

Index: chrome/browser/extensions/api/content_settings/content_settings_helpers.cc
diff --git a/chrome/browser/extensions/api/content_settings/content_settings_helpers.cc b/chrome/browser/extensions/api/content_settings/content_settings_helpers.cc
index 7730dc2da835e10ff590804c01d7f6095fdc0d96..ca8287ae6550e92a7369a1a6b4109e0735d49586 100644
--- a/chrome/browser/extensions/api/content_settings/content_settings_helpers.cc
+++ b/chrome/browser/extensions/api/content_settings/content_settings_helpers.cc
@@ -27,7 +27,10 @@ const char* const kContentSettingsTypeNames[] = {
   "notifications",
   "auto-select-certificate",
   "fullscreen",
-  "mouselock"
+  "mouselock",
+  "register-protocol-handler",

[not at google - send to devlin, 2015/01/30 20:11:52]

why are there 3 changes here (in particular RPH seems unrelated to media)

[Mike West, 2015/01/31 09:51:59]

This array needs to match the ContentSettingsType enum
(https://code.google.com/p/chromium/codesearch#chromium/src/components/content...).
I'd much prefer for us to be a bit cleverer about this (assert sameness at a
minimum (see the TODO below (and note that the enum is in a component, so direct
assertion is right out for the moment (but, really, so is the cast below..
hooray for layering violations!)))), but for the moment, this is how things
work.

[not at google - send to devlin, 2015/02/02 17:52:57]

Okie I'm a little confused, it doesn't match the enum though? RPH comes after
the media stream values from what I can tell.

[msramek, 2015/02/02 18:50:52]

Oh! Thanks, good catch :-) (I would notice it when I got to RPH, but your
confusion prevented my future confusion :-) ).

I didn't want to just invent these strings, so I followed the mapping in
|kContentSettingsTypeGroupNames| and wrote these strings in the same order as
there. The order is ALMOST the same as in |ContentSettingsType|, except one
difference that I didn't notice. The 11th entry is

MIXED_SCRIPT in |ContentSettingsType|
PROTOCOL_HANDLERS in |kContentSettingsTypeGroupNames|

And since this 11th entry was unused so far (it's the only one of them not yet
occurring in content_settings.json), it didn't matter what is its value and
tests passed anyway.

So I'm removing the reference to |kContentSettingsTypeGroupNames| from the TODO,
since that was a misstep. For consistency, it would be nice to synchronize these
strings with |kTypeNames| in content_settings_utils.cc, but it's not necessary
for this to work.

+  "media-stream",
+  "media-stream-mic"
 };
 
 // TODO(msramek): Assert that |kContentSettingsTypeNames| is synced with enum