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

Unified Diff: services/video_capture/fake_device_descriptor_unittest.cc

Issue 2379253003: Video Capture Service: typemap video_capture's and media's VideoCaptureFormat (Closed)
Patch Set: more traits methods inlining Created 4 years, 2 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 | « services/video_capture/DEPS ('k') | services/video_capture/fake_device_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: services/video_capture/fake_device_descriptor_unittest.cc
diff --git a/services/video_capture/fake_device_descriptor_unittest.cc b/services/video_capture/fake_device_descriptor_unittest.cc
index c0764270b85bbaac80af5b03565c7cb939b01419..c20c020e8b08ceb477ff8b293dba025a95a077ac 100644
--- a/services/video_capture/fake_device_descriptor_unittest.cc
+++ b/services/video_capture/fake_device_descriptor_unittest.cc
@@ -71,11 +71,11 @@ TEST_F(FakeDeviceDescriptorTest, CanUseSecondRequestedProxy) {
&wait_loop));
wait_loop.Run();
- auto arbitrary_requested_format = mojom::VideoCaptureFormat::New();
- arbitrary_requested_format->frame_size.SetSize(640, 480);
- arbitrary_requested_format->frame_rate = 15;
- arbitrary_requested_format->pixel_format = media::mojom::VideoFormat::I420;
- arbitrary_requested_format->pixel_storage = mojom::VideoPixelStorage::CPU;
+ media::VideoCaptureFormat arbitrary_requested_format;
+ arbitrary_requested_format.frame_size.SetSize(640, 480);
+ arbitrary_requested_format.frame_rate = 15;
+ arbitrary_requested_format.pixel_format = media::PIXEL_FORMAT_I420;
+ arbitrary_requested_format.pixel_storage = media::PIXEL_STORAGE_CPU;
base::RunLoop wait_loop_2;
mojom::VideoCaptureDeviceClientPtr client_proxy;
@@ -84,7 +84,7 @@ TEST_F(FakeDeviceDescriptorTest, CanUseSecondRequestedProxy) {
.WillRepeatedly(
InvokeWithoutArgs([&wait_loop_2]() { wait_loop_2.Quit(); }));
- device_proxy_2->Start(std::move(arbitrary_requested_format),
+ device_proxy_2->Start(arbitrary_requested_format,
mojom::ResolutionChangePolicy::FIXED_RESOLUTION,
mojom::PowerLineFrequency::DEFAULT,
std::move(client_proxy));
« no previous file with comments | « services/video_capture/DEPS ('k') | services/video_capture/fake_device_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698