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

Unified Diff: chrome/browser/media/webrtc/media_stream_devices_controller.h

Issue 2711883003: Change MediaStreamDevicesController tests to use RequestPermissions function (Closed)
Patch Set: Add test delegate 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
« no previous file with comments | « no previous file | chrome/browser/media/webrtc/media_stream_devices_controller.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/media/webrtc/media_stream_devices_controller.h
diff --git a/chrome/browser/media/webrtc/media_stream_devices_controller.h b/chrome/browser/media/webrtc/media_stream_devices_controller.h
index 2cc3427644e95c48a55bf23be090f820ade14623..0e50282e8a18eeb9e66d393d998db9820f1ba28a 100644
--- a/chrome/browser/media/webrtc/media_stream_devices_controller.h
+++ b/chrome/browser/media/webrtc/media_stream_devices_controller.h
@@ -69,6 +69,23 @@ class MediaStreamDevicesController : public PermissionRequest {
friend class MediaStreamDevicesControllerTest;
friend class policy::MediaStreamDevicesControllerBrowserTest;
+ // Delegate showing permission prompts.
+ class PermissionPromptDelegate {
+ public:
+ virtual void ShowPrompt(
+ bool user_gesture,
+ content::WebContents* web_contents,
+ std::unique_ptr<MediaStreamDevicesController> controller) = 0;
+ };
+
+ class PermissionPromptDelegateImpl;
+
+ static void RequestPermissionsWithDelegate(
+ content::WebContents* web_contents,
+ const content::MediaStreamRequest& request,
+ const content::MediaResponseCallback& callback,
+ PermissionPromptDelegate* delegate);
+
MediaStreamDevicesController(content::WebContents* web_contents,
const content::MediaStreamRequest& request,
const content::MediaResponseCallback& callback);
@@ -139,6 +156,8 @@ class MediaStreamDevicesController : public PermissionRequest {
// audio/video devices was granted or not.
content::MediaResponseCallback callback_;
+ std::unique_ptr<PermissionPromptDelegate> delegate_;
+
DISALLOW_COPY_AND_ASSIGN(MediaStreamDevicesController);
};
« no previous file with comments | « no previous file | chrome/browser/media/webrtc/media_stream_devices_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698