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

Unified Diff: services/video_capture/fake_device_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
Index: services/video_capture/fake_device_unittest.cc
diff --git a/services/video_capture/fake_device_unittest.cc b/services/video_capture/fake_device_unittest.cc
index 9e37731cce514f2fb14c410f2a84b658e84bdde3..6c43129e7b540051d5db136f2db876992df6a5ab 100644
--- a/services/video_capture/fake_device_unittest.cc
+++ b/services/video_capture/fake_device_unittest.cc
@@ -15,11 +15,11 @@ using testing::InvokeWithoutArgs;
namespace video_capture {
TEST_F(FakeDeviceTest, FrameCallbacksArrive) {
- 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;
const int kNumFramesToWaitFor = 3;
@@ -35,7 +35,7 @@ TEST_F(FakeDeviceTest, FrameCallbacksArrive) {
}
}));
- fake_device_proxy_->Start(std::move(arbitrary_requested_format),
+ fake_device_proxy_->Start(arbitrary_requested_format,
mojom::ResolutionChangePolicy::FIXED_RESOLUTION,
mojom::PowerLineFrequency::DEFAULT,
std::move(client_proxy));

Powered by Google App Engine
This is Rietveld 408576698