refactor to Introduce AUDIO_CAPTURE and VIDEO_CAPTURE permissions.

chrome/browser/media/media_stream_device_permission_context.cc

Index: chrome/browser/media/media_stream_device_permission_context.cc
diff --git a/chrome/browser/media/media_stream_device_permission_context.cc b/chrome/browser/media/media_stream_device_permission_context.cc
new file mode 100644
index 0000000000000000000000000000000000000000..86663000b4152aa7285441dc0f2768134d5d85cd
--- /dev/null
+++ b/chrome/browser/media/media_stream_device_permission_context.cc
@@ -0,0 +1,81 @@
+// Copyright 2015 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/browser/media/media_stream_device_permission_context.h"
+#include "chrome/browser/media/media_stream_device_permissions.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chrome/common/pref_names.h"
+#include "components/content_settings/core/browser/host_content_settings_map.h"
+#include "components/content_settings/core/common/content_settings.h"
+
+MediaStreamDevicePermissionContext::MediaStreamDevicePermissionContext(
+    Profile* profile,
+    const ContentSettingsType content_settings_type)
+    : PermissionContextBase(profile, content_settings_type),
+      content_settings_type_(content_settings_type) {
+  DCHECK(content_settings_type_ == CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC ||
+         content_settings_type_ == CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA);
+}
+
+MediaStreamDevicePermissionContext::~MediaStreamDevicePermissionContext() {}
+
+void MediaStreamDevicePermissionContext::RequestPermission(
+    content::WebContents* web_contents,
+    const PermissionRequestID& id,
+    const GURL& requesting_frame,
+    bool user_gesture,
+    const BrowserPermissionCallback& callback) {
+  NOTREACHED() << "RequestPermission is not implemented";
+  callback(CONTENT_SETTING_BLOCK);
+}
+
+ContentSetting MediaStreamDevicePermissionContext::GetPermissionStatus(
+    const GURL& requesting_origin,
+    const GURL& embedding_origin) const {
+  // TODO(raymes): Merge this policy check into content settings
+  // crbug.com/244389.
+  const char* policy_name = nullptr;
+  const char* urls_policy_name = nullptr;
+  if (content_settings_type_ == CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC) {
+    policy_name = prefs::kAudioCaptureAllowed;
+    urls_policy_name = prefs::kAudioCaptureAllowedUrls;
+  } else {
+    DCHECK(content_settings_type_ == CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA);
+    policy_name = prefs::kVideoCaptureAllowed;
+    urls_policy_name = prefs::kVideoCaptureAllowedUrls;
+  }
+
+  MediaStreamDevicePolicy policy = GetDevicePolicy(
+      profile(), requesting_origin, policy_name, urls_policy_name);
+
+  if (policy == ALWAYS_DENY)
+    return CONTENT_SETTING_BLOCK;
+
+  if (policy == ALWAYS_ALLOW)
+    return CONTENT_SETTING_ALLOW;
+
+  DCHECK_EQ(POLICY_NOT_SET, policy);
+  // Check the content setting.
+  ContentSetting setting = PermissionContextBase::GetPermissionStatus(
+      requesting_origin, embedding_origin);
+
+  return setting == CONTENT_SETTING_DEFAULT ? CONTENT_SETTING_ASK : setting;
+}
+
+void MediaStreamDevicePermissionContext::ResetPermission(
+    const GURL& requesting_origin,
+    const GURL& embedding_origin) {
+  NOTREACHED() << "ResetPermission is not implemented";
+}
+
+void MediaStreamDevicePermissionContext::CancelPermissionRequest(
+    content::WebContents* web_contents,
+    const PermissionRequestID& id) {
+  NOTREACHED() << "CancelPermissionRequest is not implemented";
+}
+
+bool MediaStreamDevicePermissionContext::IsRestrictedToSecureOrigins() const {
+  // Changing this once Flash also requires secure origin.

[palmer, 2015/08/28 01:44:21]

I'm not sure what this comment means. Does it mean that you have to return false
so that Flash can record media when the SWF is loaded from a non-secure origin?

[mlamouri (slow - plz ping), 2015/08/28 10:56:21]

+1.
Also, this is should be TODO($user) and point to a bug.

I still don't understand why we need the PermissionContext to worry about Flash
but let's discuss that in the bug.

[guoweis_left_chromium, 2015/08/28 16:16:20]

My goal here is to refactor without introducing functional change. Hence this is
probably best left with someone else to handle this. Raymes, may I delegate this
to you? Could you open a new bug to track this discussion?

+  return false;
+}