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

Unified Diff: content/browser/renderer_host/media/video_capture_host.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
Index: content/browser/renderer_host/media/video_capture_host.cc
diff --git a/content/browser/renderer_host/media/video_capture_host.cc b/content/browser/renderer_host/media/video_capture_host.cc
index bc7c8c19d7de971ad75f514774e2229af69ec8b7..f623d7d987cdaca42c3f0ec9b50c47ebfde5441d 100644
--- a/content/browser/renderer_host/media/video_capture_host.cc
+++ b/content/browser/renderer_host/media/video_capture_host.cc
@@ -95,11 +95,11 @@ void VideoCaptureHost::OnFrameInfoChanged(
const VideoCaptureControllerID& controller_id,
int width,
int height,
- int frame_per_second) {
+ int frame_rate) {
BrowserThread::PostTask(
BrowserThread::IO, FROM_HERE,
base::Bind(&VideoCaptureHost::DoSendFrameInfoChangedOnIOThread,
- this, controller_id, width, height, frame_per_second));
+ this, controller_id, width, height, frame_rate));
}
void VideoCaptureHost::OnEnded(const VideoCaptureControllerID& controller_id) {
@@ -170,7 +170,7 @@ void VideoCaptureHost::DoSendFrameInfoOnIOThread(
media::VideoCaptureParams params;
params.width = format.width;
params.height = format.height;
- params.frame_per_second = format.frame_rate;
+ params.frame_rate = format.frame_rate;
params.frame_size_type = format.frame_size_type;
Send(new VideoCaptureMsg_DeviceInfo(controller_id.device_id, params));
Send(new VideoCaptureMsg_StateChanged(controller_id.device_id,
@@ -181,7 +181,7 @@ void VideoCaptureHost::DoSendFrameInfoChangedOnIOThread(
const VideoCaptureControllerID& controller_id,
int width,
int height,
- int frame_per_second) {
+ int frame_rate) {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
if (entries_.find(controller_id) == entries_.end())
@@ -190,7 +190,7 @@ void VideoCaptureHost::DoSendFrameInfoChangedOnIOThread(
media::VideoCaptureParams params;
params.width = width;
params.height = height;
- params.frame_per_second = frame_per_second;
+ params.frame_rate = frame_rate;
Send(new VideoCaptureMsg_DeviceInfoChanged(controller_id.device_id, params));
}
@@ -215,7 +215,7 @@ void VideoCaptureHost::OnStartCapture(int device_id,
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
DVLOG(1) << "VideoCaptureHost::OnStartCapture, device_id " << device_id
<< ", (" << params.width << ", " << params.height << ", "
- << params.frame_per_second << ", " << params.session_id
+ << params.frame_rate << ", " << params.session_id
<< ", variable resolution device:"
<< ((params.frame_size_type ==
media::VariableResolutionVideoCaptureDevice) ? "yes" : "no")

Powered by Google App Engine
This is Rietveld 408576698