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

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

Issue 1125333003: Remove CheckMediaAccessPermission() from ContentBrowserClient. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 7 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_capture_devices_dispatcher.cc
diff --git a/chrome/browser/media/media_capture_devices_dispatcher.cc b/chrome/browser/media/media_capture_devices_dispatcher.cc
index 851b456bfdc85dc0aa8a407902dd27285ad1db21..79c93f5c1124def7b7aa119c90259682a0ae0532 100644
--- a/chrome/browser/media/media_capture_devices_dispatcher.cc
+++ b/chrome/browser/media/media_capture_devices_dispatcher.cc
@@ -234,21 +234,6 @@ gfx::NativeWindow FindParentWindowForWebContents(
}
#endif
-#if defined(ENABLE_EXTENSIONS)
-const extensions::Extension* GetExtensionForOrigin(
- Profile* profile,
- const GURL& security_origin) {
- if (!security_origin.SchemeIs(extensions::kExtensionScheme))
- return NULL;
-
- const extensions::Extension* extension =
- extensions::ExtensionRegistry::Get(profile)->enabled_extensions().GetByID(
- security_origin.host());
- DCHECK(extension);
- return extension;
-}
-#endif
-
} // namespace
MediaCaptureDevicesDispatcher::PendingAccessRequest::PendingAccessRequest(
@@ -369,63 +354,6 @@ void MediaCaptureDevicesDispatcher::ProcessMediaAccessRequest(
}
bool MediaCaptureDevicesDispatcher::CheckMediaAccessPermission(
- content::BrowserContext* browser_context,
- const GURL& security_origin,
- content::MediaStreamType type) {
- DCHECK_CURRENTLY_ON(BrowserThread::UI);
- DCHECK(type == content::MEDIA_DEVICE_AUDIO_CAPTURE ||
- type == content::MEDIA_DEVICE_VIDEO_CAPTURE);
-
- Profile* profile = Profile::FromBrowserContext(browser_context);
-#if defined(ENABLE_EXTENSIONS)
- const extensions::Extension* extension =
- GetExtensionForOrigin(profile, security_origin);
-
- if (extension && (extension->is_platform_app() ||
- IsMediaRequestWhitelistedForExtension(extension))) {
- return extension->permissions_data()->HasAPIPermission(
- type == content::MEDIA_DEVICE_AUDIO_CAPTURE
- ? extensions::APIPermission::kAudioCapture
- : extensions::APIPermission::kVideoCapture);
- }
-#endif
-
- ContentSettingsType contentSettingsType =
- type == content::MEDIA_DEVICE_AUDIO_CAPTURE
- ? CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC
- : CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA;
-
- if (CheckAllowAllMediaStreamContentForOrigin(
- profile, security_origin, contentSettingsType)) {
- return true;
- }
-
- const char* policy_name = type == content::MEDIA_DEVICE_AUDIO_CAPTURE
- ? prefs::kAudioCaptureAllowed
- : prefs::kVideoCaptureAllowed;
- const char* list_policy_name = type == content::MEDIA_DEVICE_AUDIO_CAPTURE
- ? prefs::kAudioCaptureAllowedUrls
- : prefs::kVideoCaptureAllowedUrls;
- if (GetDevicePolicy(
- profile, security_origin, policy_name, list_policy_name) ==
- ALWAYS_ALLOW) {
- return true;
- }
-
- // There's no secondary URL for these content types, hence duplicating
- // |security_origin|.
- if (profile->GetHostContentSettingsMap()->GetContentSetting(
- security_origin,
- security_origin,
- contentSettingsType,
- content_settings::ResourceIdentifier()) == CONTENT_SETTING_ALLOW) {
- return true;
- }
-
- return false;
-}
-
-bool MediaCaptureDevicesDispatcher::CheckMediaAccessPermission(
content::WebContents* web_contents,
const GURL& security_origin,
content::MediaStreamType type) {
« no previous file with comments | « chrome/browser/media/media_capture_devices_dispatcher.h ('k') | content/public/browser/content_browser_client.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698