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

Unified Diff: media/gpu/vaapi_video_encode_accelerator.cc

Issue 2598313002: Use GetProfileName() for error logs in gpu_video_decode_accelerator.cc (Closed)
Patch Set: Use GetProfileName() to better explain logs displaying VideoCodecProfile. Created 3 years, 11 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 | « media/gpu/vaapi_video_decode_accelerator.cc ('k') | media/gpu/video_decode_accelerator_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/gpu/vaapi_video_encode_accelerator.cc
diff --git a/media/gpu/vaapi_video_encode_accelerator.cc b/media/gpu/vaapi_video_encode_accelerator.cc
index 6505bb577fb08665f27a36a146eb67b60e40021c..bf77ad7928e22404d23e3dd361bf64beb5eed6d5 100644
--- a/media/gpu/vaapi_video_encode_accelerator.cc
+++ b/media/gpu/vaapi_video_encode_accelerator.cc
@@ -167,7 +167,7 @@ bool VaapiVideoEncodeAccelerator::Initialize(
DVLOGF(1) << "Initializing VAVEA, input_format: "
<< VideoPixelFormatToString(format)
<< ", input_visible_size: " << input_visible_size.ToString()
- << ", output_profile: " << output_profile
+ << ", output_profile: " << GetProfileName(output_profile)
<< ", initial_bitrate: " << initial_bitrate;
client_ptr_factory_.reset(new base::WeakPtrFactory<Client>(client));
@@ -179,7 +179,8 @@ bool VaapiVideoEncodeAccelerator::Initialize(
return profile.profile == output_profile;
});
if (profile == profiles.end()) {
- DVLOGF(1) << "Unsupported output profile " << output_profile;
+ DVLOGF(1) << "Unsupported output profile "
+ << GetProfileName(output_profile);
return false;
}
if (input_visible_size.width() > profile->max_resolution.width() ||
@@ -212,7 +213,8 @@ bool VaapiVideoEncodeAccelerator::Initialize(
VaapiWrapper::CreateForVideoCodec(VaapiWrapper::kEncode, output_profile,
base::Bind(&ReportToUMA, VAAPI_ERROR));
if (!vaapi_wrapper_.get()) {
- DVLOGF(1) << "Failed initializing VAAPI for profile " << output_profile;
+ DVLOGF(1) << "Failed initializing VAAPI for profile "
+ << GetProfileName(output_profile);
return false;
}
« no previous file with comments | « media/gpu/vaapi_video_decode_accelerator.cc ('k') | media/gpu/video_decode_accelerator_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698