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

Unified Diff: content/browser/renderer_host/media/video_capture_browsertest.cc

Issue 2753073006: [Mojo Video Capture] Add support to BuildableVideoCaptureDevice for aborting the device start. (Closed)
Patch Set: Incorporate suggestions from Patch Set 1 Created 3 years, 9 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: content/browser/renderer_host/media/video_capture_browsertest.cc
diff --git a/content/browser/renderer_host/media/video_capture_browsertest.cc b/content/browser/renderer_host/media/video_capture_browsertest.cc
index 47e0b274a353d7f6ad50954590711c5e957895f1..5263312ff3b776b7073465d4a68e076c1ee4958d 100644
--- a/content/browser/renderer_host/media/video_capture_browsertest.cc
+++ b/content/browser/renderer_host/media/video_capture_browsertest.cc
@@ -101,9 +101,9 @@ class VideoCaptureBrowserTest
return;
}
- video_capture_manager_->StopCaptureForClient(
- controller_.get(), stub_client_id_, &mock_controller_event_handler_,
- false);
+ video_capture_manager_->DisconnectClient(controller_.get(), stub_client_id_,
+ &mock_controller_event_handler_,
+ false);
EXPECT_CALL(mock_stream_provider_listener_, Closed(_, _))
.WillOnce(InvokeWithoutArgs([continuation]() { continuation.Run(); }));
@@ -148,7 +148,7 @@ class VideoCaptureBrowserTest
capture_params.requested_format = media::VideoCaptureFormat(
GetParam().resolution_to_use, GetParam().frame_rate_to_use,
GetParam().pixel_format_to_use);
- video_capture_manager_->StartCaptureForClient(
+ video_capture_manager_->ConnectClient(
session_id_, capture_params, stub_client_id_,
&mock_controller_event_handler_,
base::Bind(&VideoCaptureBrowserTest::OnConnectClientToControllerAnswer,

Powered by Google App Engine
This is Rietveld 408576698