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

Unified Diff: media/video/capture/video_capture_device_unittest.cc

Issue 22876027: Consolidate duplicated frame / capability structures in video_capture_types.h (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 4 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: media/video/capture/video_capture_device_unittest.cc
diff --git a/media/video/capture/video_capture_device_unittest.cc b/media/video/capture/video_capture_device_unittest.cc
index e39c59b0541c76db88513d72b01d51e23e0e274c..237f941f700fcfbb434e9bbb6ee4e2caec56a845 100644
--- a/media/video/capture/video_capture_device_unittest.cc
+++ b/media/video/capture/video_capture_device_unittest.cc
@@ -159,7 +159,7 @@ TEST_F(VideoCaptureDeviceTest, CaptureVGA) {
VideoCaptureCapability capture_format(640,
480,
30,
- VideoCaptureCapability::kI420,
+ kI420,
0,
false,
ConstantResolutionVideoCaptureDevice);
@@ -197,7 +197,7 @@ TEST_F(VideoCaptureDeviceTest, Capture720p) {
VideoCaptureCapability capture_format(1280,
720,
30,
- VideoCaptureCapability::kI420,
+ kI420,
0,
false,
ConstantResolutionVideoCaptureDevice);
@@ -231,7 +231,7 @@ TEST_F(VideoCaptureDeviceTest, MAYBE_AllocateBadSize) {
VideoCaptureCapability capture_format(637,
472,
35,
- VideoCaptureCapability::kI420,
+ kI420,
0,
false,
ConstantResolutionVideoCaptureDevice);
@@ -258,21 +258,21 @@ TEST_F(VideoCaptureDeviceTest, ReAllocateCamera) {
VideoCaptureCapability capture_format_1(640,
480,
30,
- VideoCaptureCapability::kI420,
+ kI420,
0,
false,
ConstantResolutionVideoCaptureDevice);
VideoCaptureCapability capture_format_2(1280,
1024,
30,
- VideoCaptureCapability::kI420,
+ kI420,
0,
false,
ConstantResolutionVideoCaptureDevice);
VideoCaptureCapability capture_format_3(320,
240,
30,
- VideoCaptureCapability::kI420,
+ kI420,
0,
false,
ConstantResolutionVideoCaptureDevice);
@@ -321,7 +321,7 @@ TEST_F(VideoCaptureDeviceTest, DeAllocateCameraWhileRunning) {
VideoCaptureCapability capture_format(640,
480,
30,
- VideoCaptureCapability::kI420,
+ kI420,
0,
false,
ConstantResolutionVideoCaptureDevice);
@@ -359,7 +359,7 @@ TEST_F(VideoCaptureDeviceTest, FakeCapture) {
VideoCaptureCapability capture_format(640,
480,
30,
- VideoCaptureCapability::kI420,
+ kI420,
0,
false,
ConstantResolutionVideoCaptureDevice);
@@ -396,7 +396,7 @@ TEST_F(VideoCaptureDeviceTest, MAYBE_CaptureMjpeg) {
VideoCaptureCapability capture_format(1280,
720,
30,
- VideoCaptureCapability::kMJPEG,
+ kMJPEG,
0,
false,
ConstantResolutionVideoCaptureDevice);
@@ -406,7 +406,7 @@ TEST_F(VideoCaptureDeviceTest, MAYBE_CaptureMjpeg) {
// Get captured video frames.
PostQuitTask();
EXPECT_TRUE(wait_event_.TimedWait(TestTimeouts::action_max_timeout()));
- EXPECT_EQ(rx_capability.color, VideoCaptureCapability::kMJPEG);
+ EXPECT_EQ(rx_capability.color, kMJPEG);
device->DeAllocate();
}

Powered by Google App Engine
This is Rietveld 408576698