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

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

Issue 489183003: Log error messages in windows video capture (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: refactor log message to os independendent Created 6 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/win/video_capture_device_factory_win.cc
diff --git a/media/video/capture/win/video_capture_device_factory_win.cc b/media/video/capture/win/video_capture_device_factory_win.cc
index 75ee59d3bf495be332468550d2bb9ef22a16ef5d..4a970a80935575a3eebdb44816504e811b525781 100644
--- a/media/video/capture/win/video_capture_device_factory_win.cc
+++ b/media/video/capture/win/video_capture_device_factory_win.cc
@@ -186,7 +186,7 @@ static void GetDeviceNamesMediaFoundation(
}
}
if (FAILED(hr))
- DLOG(WARNING) << "GetAllocatedString failed: " << std::hex << hr;
+ DPLOG(ERROR) << "Failed to GetAllocatedString";
tommi (sloooow) - chröme 2014/08/27 14:46:18 FYI there's also DPLOG_IF. Here you could remove
magjed_chromium 2014/08/27 17:03:27 Nice. The arguments are reversed though.
devices[i]->Release();
}
}
@@ -216,7 +216,7 @@ static void GetDeviceSupportedFormatsDirectShow(
hr = VideoCaptureDeviceWin::GetDeviceFilter(device,
capture_filter.Receive());
if (!capture_filter) {
- DVLOG(2) << "Failed to create capture filter.";
+ DPLOG(ERROR) << "Failed to create capture filter";
return;
}
@@ -225,14 +225,14 @@ static void GetDeviceSupportedFormatsDirectShow(
PINDIR_OUTPUT,
PIN_CATEGORY_CAPTURE));
if (!output_capture_pin) {
- DVLOG(2) << "Failed to get capture output pin";
+ DPLOG(ERROR) << "Failed to get capture output pin";
return;
}
ScopedComPtr<IAMStreamConfig> stream_config;
hr = output_capture_pin.QueryInterface(stream_config.Receive());
if (FAILED(hr)) {
- DVLOG(2) << "Failed to get IAMStreamConfig interface from "
+ DPLOG(ERROR) << "Failed to get IAMStreamConfig interface from "
"capture device";
return;
}
@@ -240,7 +240,7 @@ static void GetDeviceSupportedFormatsDirectShow(
int count = 0, size = 0;
hr = stream_config->GetNumberOfCapabilities(&count, &size);
if (FAILED(hr)) {
- DVLOG(2) << "Failed to GetNumberOfCapabilities";
+ DPLOG(ERROR) << "Failed to GetNumberOfCapabilities";
return;
}
@@ -251,7 +251,7 @@ static void GetDeviceSupportedFormatsDirectShow(
// GetStreamCaps() may return S_FALSE, so don't use FAILED() or SUCCEED()
// macros here since they'll trigger incorrectly.
if (hr != S_OK) {
- DVLOG(2) << "Failed to GetStreamCaps";
+ DPLOG(ERROR) << "Failed to GetStreamCaps";
return;
}
@@ -293,7 +293,7 @@ static void GetDeviceSupportedFormatsMediaFoundation(
HRESULT hr =
MFCreateSourceReaderFromMediaSource(source, NULL, reader.Receive());
if (FAILED(hr)) {
- DLOG(ERROR) << "MFCreateSourceReaderFromMediaSource: " << std::hex << hr;
+ DPLOG(ERROR) << "Failed to MFCreateSourceReaderFromMediaSource";
return;
}
@@ -307,7 +307,7 @@ static void GetDeviceSupportedFormatsMediaFoundation(
UINT32 width, height;
hr = MFGetAttributeSize(type, MF_MT_FRAME_SIZE, &width, &height);
if (FAILED(hr)) {
- DLOG(ERROR) << "MFGetAttributeSize: " << std::hex << hr;
+ DPLOG(ERROR) << "Failed to MFGetAttributeSize";
return;
}
VideoCaptureFormat capture_format;
@@ -316,7 +316,7 @@ static void GetDeviceSupportedFormatsMediaFoundation(
UINT32 numerator, denominator;
hr = MFGetAttributeRatio(type, MF_MT_FRAME_RATE, &numerator, &denominator);
if (FAILED(hr)) {
- DLOG(ERROR) << "MFGetAttributeSize: " << std::hex << hr;
+ DPLOG(ERROR) << "Failed to MFGetAttributeSize";
return;
}
capture_format.frame_rate = denominator
@@ -325,7 +325,7 @@ static void GetDeviceSupportedFormatsMediaFoundation(
GUID type_guid;
hr = type->GetGUID(MF_MT_SUBTYPE, &type_guid);
if (FAILED(hr)) {
- DLOG(ERROR) << "GetGUID: " << std::hex << hr;
+ DPLOG(ERROR) << "Failed to GetGUID";
return;
}
VideoCaptureDeviceMFWin::FormatFromGuid(type_guid,

Powered by Google App Engine
This is Rietveld 408576698