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

Unified Diff: media/gpu/vt_video_encode_accelerator_mac.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/video_encode_accelerator_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/gpu/vt_video_encode_accelerator_mac.cc
diff --git a/media/gpu/vt_video_encode_accelerator_mac.cc b/media/gpu/vt_video_encode_accelerator_mac.cc
index cc5ed0d0b8e8a7e896e03754cfdfab3343bfdf88..f5aff7e4c381930a28278b924844c62e7c518bc6 100644
--- a/media/gpu/vt_video_encode_accelerator_mac.cc
+++ b/media/gpu/vt_video_encode_accelerator_mac.cc
@@ -123,7 +123,7 @@ bool VTVideoEncodeAccelerator::Initialize(VideoPixelFormat format,
Client* client) {
DVLOG(3) << __func__ << ": input_format=" << VideoPixelFormatToString(format)
<< ", input_visible_size=" << input_visible_size.ToString()
- << ", output_profile=" << output_profile
+ << ", output_profile=" << GetProfileName(output_profile)
<< ", initial_bitrate=" << initial_bitrate;
DCHECK(thread_checker_.CalledOnValidThread());
DCHECK(client);
@@ -134,7 +134,8 @@ bool VTVideoEncodeAccelerator::Initialize(VideoPixelFormat format,
return false;
}
if (H264PROFILE_BASELINE != output_profile) {
- DLOG(ERROR) << "Output profile not supported= " << output_profile;
+ DLOG(ERROR) << "Output profile not supported= "
+ << GetProfileName(output_profile);
return false;
}
« no previous file with comments | « media/gpu/video_encode_accelerator_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698