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

Unified Diff: media/video/capture/win/video_capture_device_win.cc

Issue 10035054: Merge VideoCaptureDevice::Capability with media::VideoCaptureCapability (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Created 8 years, 8 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/win/video_capture_device_win.cc
diff --git a/media/video/capture/win/video_capture_device_win.cc b/media/video/capture/win/video_capture_device_win.cc
index d073d21beea71a21e87e5305eaa10dc919bc3b04..c973e1ff73c28e115538112981bbb25f31b96ccb 100644
--- a/media/video/capture/win/video_capture_device_win.cc
+++ b/media/video/capture/win/video_capture_device_win.cc
@@ -148,7 +148,7 @@ struct ResolutionDiff {
int diff_height;
int diff_width;
int diff_frame_rate;
- media::VideoCaptureDevice::Format color;
+ media::VideoFrame::Format color;
};
bool CompareHeight(const ResolutionDiff& item1, const ResolutionDiff& item2) {
@@ -338,7 +338,7 @@ void VideoCaptureDeviceWin::Allocate(
// Get the camera capability that best match the requested resolution.
const int capability_index = GetBestMatchedCapability(width, height,
frame_rate);
- Capability capability = capabilities_[capability_index];
+ media::VideoCaptureCapability capability = capabilities_[capability_index];
// Reduce the frame rate if the requested frame rate is lower
// than the capability.
@@ -373,7 +373,7 @@ void VideoCaptureDeviceWin::Allocate(
if (FAILED(hr))
SetErrorState("Failed to set capture device output format");
- if (capability.color == VideoCaptureDevice::kMJPEG && !mjpg_filter_.get()) {
+ if (capability.color == media::VideoFrame::kMJPEG && !mjpg_filter_.get()) {
// Create MJPG filter if we need it.
hr = mjpg_filter_.CreateInstance(CLSID_MjpegDec, NULL, CLSCTX_INPROC);
@@ -391,7 +391,7 @@ void VideoCaptureDeviceWin::Allocate(
}
}
- if (capability.color == VideoCaptureDevice::kMJPEG && mjpg_filter_.get()) {
+ if (capability.color == media::VideoFrame::kMJPEG && mjpg_filter_.get()) {
// Connect the camera to the MJPEG decoder.
hr = graph_builder_->ConnectDirect(output_capture_pin_, input_mjpg_pin_,
NULL);
@@ -417,7 +417,8 @@ void VideoCaptureDeviceWin::Allocate(
// Get the capability back from the sink filter after the filter have been
// connected.
- const Capability& used_capability = sink_filter_->ResultingCapability();
+ const media::VideoCaptureCapability& used_capability =
+ sink_filter_->ResultingCapability();
observer_->OnFrameInfo(used_capability);
state_ = kAllocated;
@@ -515,7 +516,7 @@ bool VideoCaptureDeviceWin::CreateCapabilityMap() {
if (media_type->majortype == MEDIATYPE_Video &&
media_type->formattype == FORMAT_VideoInfo) {
- Capability capability;
+ media::VideoCaptureCapability capability;
REFERENCE_TIME time_per_frame = 0;
VIDEOINFOHEADER* h =
@@ -559,16 +560,16 @@ bool VideoCaptureDeviceWin::CreateCapabilityMap() {
// We can't switch MEDIATYPE :~(.
if (media_type->subtype == kMediaSubTypeI420) {
- capability.color = VideoCaptureDevice::kI420;
+ capability.color = media::VideoFrame::kI420;
} else if (media_type->subtype == MEDIASUBTYPE_IYUV) {
// This is identical to kI420.
- capability.color = VideoCaptureDevice::kI420;
+ capability.color = media::VideoFrame::kI420;
} else if (media_type->subtype == MEDIASUBTYPE_RGB24) {
- capability.color = VideoCaptureDevice::kRGB24;
+ capability.color = media::VideoFrame::kRGB24;
} else if (media_type->subtype == MEDIASUBTYPE_YUY2) {
- capability.color = VideoCaptureDevice::kYUY2;
+ capability.color = media::VideoFrame::kYUY2;
} else if (media_type->subtype == MEDIASUBTYPE_MJPG) {
- capability.color = VideoCaptureDevice::kMJPEG;
+ capability.color = media::VideoFrame::kMJPEG;
} else {
WCHAR guid_str[128];
StringFromGUID2(media_type->subtype, guid_str, arraysize(guid_str));
@@ -598,7 +599,7 @@ int VideoCaptureDeviceWin::GetBestMatchedCapability(int requested_width,
for (CapabilityMap::iterator iterator = capabilities_.begin();
iterator != capabilities_.end();
++iterator) {
- Capability capability = iterator->second;
+ media::VideoCaptureCapability capability = iterator->second;
ResolutionDiff diff;
diff.capability_index = iterator->first;

Powered by Google App Engine
This is Rietveld 408576698