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

Unified Diff: chrome/browser/media/webrtc/media_stream_devices_controller_browsertest.cc

Issue 2811913002: Pull code associated with a Media PermissionRequest out of MediaStreamDevicesController (Closed)
Patch Set: MediaRequest Created 3 years, 8 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/webrtc/media_stream_devices_controller_browsertest.cc
diff --git a/chrome/browser/media/webrtc/media_stream_devices_controller_browsertest.cc b/chrome/browser/media/webrtc/media_stream_devices_controller_browsertest.cc
index 7526eeefd9e901ad05c8ee1e1d980d262f587be0..19d44472321bff4fa15679ba9141ed6de6b1cfa5 100644
--- a/chrome/browser/media/webrtc/media_stream_devices_controller_browsertest.cc
+++ b/chrome/browser/media/webrtc/media_stream_devices_controller_browsertest.cc
@@ -36,21 +36,21 @@ class MediaStreamDevicesControllerTest : public WebRtcTestBase {
// we should remove PermissionPromptDelegate and just use
// MockPermissionPromptFactory instead. The APIs are the same.
class TestPermissionPromptDelegate
- : public internal::PermissionPromptDelegate {
+ : public MediaStreamDevicesController::PermissionPromptDelegate {
public:
- void ShowPrompt(
- bool user_gesture,
- content::WebContents* web_contents,
- std::unique_ptr<MediaStreamDevicesController> controller) override {
- if (controller->IsAskingForAudio())
+ void ShowPrompt(bool user_gesture,
+ content::WebContents* web_contents,
+ std::unique_ptr<MediaStreamDevicesController::Request>
+ request) override {
+ if (request->IsAskingForAudio())
last_requests_.push_back(CONTENT_SETTINGS_TYPE_MEDIASTREAM_MIC);
- if (controller->IsAskingForVideo())
+ if (request->IsAskingForVideo())
last_requests_.push_back(CONTENT_SETTINGS_TYPE_MEDIASTREAM_CAMERA);
if (response_type_ == PermissionRequestManager::ACCEPT_ALL)
- controller->PermissionGranted();
+ request->PermissionGranted();
else if (response_type_ == PermissionRequestManager::DENY_ALL)
- controller->PermissionDenied();
+ request->PermissionDenied();
}
void set_response_type(

Powered by Google App Engine
This is Rietveld 408576698