Stop using the MEDIASTREAM content setting.

chrome/browser/media/media_stream_device_permissions.cc

Index: chrome/browser/media/media_stream_device_permissions.cc
diff --git a/chrome/browser/media/media_stream_device_permissions.cc b/chrome/browser/media/media_stream_device_permissions.cc
index c8ff9a36b86d3ee8998d9f45b3c3e47a2cb34abb..6ce1c7b989de25ab2bec226f30047a4de5fb6dec 100644
--- a/chrome/browser/media/media_stream_device_permissions.cc
+++ b/chrome/browser/media/media_stream_device_permissions.cc
@@ -36,12 +36,12 @@ bool IsInKioskMode() {
 }  // namespace
 
 bool CheckAllowAllMediaStreamContentForOrigin(Profile* profile,

[markusheintz_, 2015/03/06 16:05:33]

Do we actually need this method anymore? Seems like one step of indirection we
could get rid of.

[msramek, 2015/03/06 17:47:33]

Not really, no. I just didn't want to introduce or remove any layers to make
this CL more readable. Basically, so that all replacements are context-free.

Do you mind if I leave this kind of cleanup for a follow-up CL?

-                                              const GURL& security_origin) {
-  // TODO(markusheintz): Replace CONTENT_SETTINGS_TYPE_MEDIA_STREAM with the
-  // appropriate new CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC and
-  // CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA.
+                                              const GURL& security_origin,
+                                              ContentSettingsType type) {
+  DCHECK(type == CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC ||
+         type == CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA);
   return profile->GetHostContentSettingsMap()->ShouldAllowAllContent(
-      security_origin, security_origin, CONTENT_SETTINGS_TYPE_MEDIASTREAM);
+      security_origin, security_origin, type);
 }
 
 MediaStreamDevicePolicy GetDevicePolicy(Profile* profile,