Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(678)

Unified Diff: chrome/browser/media/media_stream_device_permission_context.cc

Issue 2307083002: Cleanup: move WebRTC related files from chrome/browser/media to chrome/browser/media/webrtc/ (Closed)
Patch Set: Removed file wrongly resuscitated during rebase Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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
deleted file mode 100644
index 43238171068eb8380f523c97e24c5b43cf51dc49..0000000000000000000000000000000000000000
--- a/chrome/browser/media/media_stream_device_permission_context.cc
+++ /dev/null
@@ -1,92 +0,0 @@
-// 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"
-#include "content/public/common/url_constants.h"
-#include "extensions/common/constants.h"
-
-MediaStreamDevicePermissionContext::MediaStreamDevicePermissionContext(
- Profile* profile,
- const content::PermissionType permission_type,
- const ContentSettingsType content_settings_type)
- : PermissionContextBase(profile, permission_type, 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.Run(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);
-
- switch (policy) {
- case ALWAYS_DENY:
- return CONTENT_SETTING_BLOCK;
- case ALWAYS_ALLOW:
- return CONTENT_SETTING_ALLOW;
- default:
- DCHECK_EQ(POLICY_NOT_SET, policy);
- }
-
- // Check the content setting. TODO(raymes): currently mic/camera permission
- // doesn't consider the embedder.
- ContentSetting setting = PermissionContextBase::GetPermissionStatus(
- requesting_origin, requesting_origin);
-
- if (setting == CONTENT_SETTING_DEFAULT)
- setting = CONTENT_SETTING_ASK;
-
- return 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 {
- // Flash currently doesn't require secure origin to use mic/camera. If we
- // return true here, it'll break the use case like http://tinychat.com.
- // TODO(raymes): Change this to true after crbug.com/526324 is fixed.
- return false;
-}

Powered by Google App Engine
This is Rietveld 408576698