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

Unified Diff: content/browser/renderer_host/media/service_video_capture_device_launcher.h

Issue 2886303002: [Mojo Video Capture] Add unit tests for ServiceVideoCaptureDeviceLauncher (Closed)
Patch Set: Incorporated suggestions from Patch Set 2 Created 3 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: content/browser/renderer_host/media/service_video_capture_device_launcher.h
diff --git a/content/browser/renderer_host/media/service_video_capture_device_launcher.h b/content/browser/renderer_host/media/service_video_capture_device_launcher.h
index 31f1c21d3a12bfca15ce8db011390f0f5394ea37..30fcf3169bfab3e8349d8aa411eaed5cabc7b381 100644
--- a/content/browser/renderer_host/media/service_video_capture_device_launcher.h
+++ b/content/browser/renderer_host/media/service_video_capture_device_launcher.h
@@ -13,7 +13,8 @@ namespace content {
// Implementation of VideoCaptureDeviceLauncher that uses the "video_capture"
// service.
-class ServiceVideoCaptureDeviceLauncher : public VideoCaptureDeviceLauncher {
+class CONTENT_EXPORT ServiceVideoCaptureDeviceLauncher
+ : public VideoCaptureDeviceLauncher {
public:
explicit ServiceVideoCaptureDeviceLauncher(
video_capture::mojom::DeviceFactoryPtr* device_factory);
@@ -41,13 +42,15 @@ class ServiceVideoCaptureDeviceLauncher : public VideoCaptureDeviceLauncher {
const media::VideoCaptureParams& params,
video_capture::mojom::DevicePtr device,
base::WeakPtr<media::VideoFrameReceiver> receiver,
- Callbacks* callbacks,
- base::OnceClosure done_cb,
video_capture::mojom::DeviceAccessResultCode result_code);
+ void OnConnectionLostWhileWaitingForCallback();
+
video_capture::mojom::DeviceFactoryPtr* const device_factory_;
State state_;
base::SequenceChecker sequence_checker_;
+ base::OnceClosure done_cb_;
+ Callbacks* callbacks_;
};
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698