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

Unified Diff: content/renderer/media/video_capture_impl.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: Rebased: changed media::VideoCaptureCapability::kI420 -> media::PIXEL_FORMAT_I420 etc. Created 7 years, 3 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 | « content/renderer/media/video_capture_impl.h ('k') | content/renderer/media/video_capture_impl_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/media/video_capture_impl.cc
diff --git a/content/renderer/media/video_capture_impl.cc b/content/renderer/media/video_capture_impl.cc
index 032cd7fedbba1a72f0541340bbdd3ccb2c2ce4a5..2dc8fabe595d50a734cecd5bb78144b65d2b3e79 100644
--- a/content/renderer/media/video_capture_impl.cc
+++ b/content/renderer/media/video_capture_impl.cc
@@ -55,7 +55,7 @@ VideoCaptureImpl::VideoCaptureImpl(
capture_message_loop_proxy_(capture_message_loop_proxy),
io_message_loop_proxy_(ChildProcess::current()->io_message_loop_proxy()),
device_id_(0),
- video_type_(media::VideoCaptureCapability::kI420),
+ video_type_(media::PIXEL_FORMAT_I420),
device_info_available_(false),
suspended_(false),
state_(VIDEO_CAPTURE_STATE_STOPPED) {
@@ -86,7 +86,7 @@ void VideoCaptureImpl::DeInit(base::Closure task) {
void VideoCaptureImpl::StartCapture(
media::VideoCapture::EventHandler* handler,
const media::VideoCaptureCapability& capability) {
- DCHECK_EQ(capability.color, media::VideoCaptureCapability::kI420);
+ DCHECK_EQ(capability.color, media::PIXEL_FORMAT_I420);
capture_message_loop_proxy_->PostTask(FROM_HERE,
base::Bind(&VideoCaptureImpl::DoStartCaptureOnCaptureThread,
@@ -438,7 +438,7 @@ void VideoCaptureImpl::StartCaptureInternal() {
media::VideoCaptureParams capability_as_params_copy;
capability_as_params_copy.width = capture_format_.width;
capability_as_params_copy.height = capture_format_.height;
- capability_as_params_copy.frame_per_second = capture_format_.frame_rate;
+ capability_as_params_copy.frame_rate = capture_format_.frame_rate;
capability_as_params_copy.session_id = capture_format_.session_id;
capability_as_params_copy.frame_size_type = capture_format_.frame_size_type;
Send(new VideoCaptureHostMsg_Start(device_id_, capability_as_params_copy));
« no previous file with comments | « content/renderer/media/video_capture_impl.h ('k') | content/renderer/media/video_capture_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698