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

Unified Diff: chrome/browser/ui/app_list/start_page_service.cc

Issue 2706813004: Move logic to show permission prompts into MediaStreamDevicesController (Closed)
Patch Set: Created 3 years, 10 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/ui/app_list/start_page_service.cc
diff --git a/chrome/browser/ui/app_list/start_page_service.cc b/chrome/browser/ui/app_list/start_page_service.cc
index 5e7d5052d20f4db533636e7c0e4257cdec3792a0..f75493ab6280310727aeae01bbb597825e8ed8b4 100644
--- a/chrome/browser/ui/app_list/start_page_service.cc
+++ b/chrome/browser/ui/app_list/start_page_service.cc
@@ -163,9 +163,8 @@ class StartPageService::StartPageWebContentsDelegate
content::WebContents* web_contents,
const content::MediaStreamRequest& request,
const content::MediaResponseCallback& callback) override {
- MediaStreamDevicesController controller(web_contents, request, callback);
- if (controller.IsAskingForVideo() || controller.IsAskingForAudio())
- NOTREACHED() << "Media stream not allowed for WebUI";
+ MediaStreamDevicesController::RequestPermissions(web_contents, request,
+ callback);
}
bool CheckMediaAccessPermission(content::WebContents* web_contents,

Powered by Google App Engine
This is Rietveld 408576698