From 3b4dd4c71aea94bb800fcc4a694379129d87c6f9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Henrik=20Bostr=C3=B6m?= Date: Mon, 5 Apr 2021 10:23:31 +0200 Subject: [PATCH] LibvpxVp8Encoder: Clarify RTC_LOG error message. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit While debugging https://crbug.com/1195144 I found it useful to clarify this log statement. The log would say "When scaling [kNative], the image was unexpectedly converted to [kI420]..." but not saying what it was trying to convert it to. This CL adds: "... instead of [kNV12]." Bug: chromium:1195144 Change-Id: I13e0040edf5d7d98d80ce674812f67dfb73be36e Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/214040 Reviewed-by: Ilya Nikolaevskiy Commit-Queue: Henrik Boström Cr-Commit-Position: refs/heads/master@{#33634} --- modules/video_coding/codecs/vp8/libvpx_vp8_encoder.cc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/modules/video_coding/codecs/vp8/libvpx_vp8_encoder.cc b/modules/video_coding/codecs/vp8/libvpx_vp8_encoder.cc index 414b5333c3..2411c1622e 100644 --- a/modules/video_coding/codecs/vp8/libvpx_vp8_encoder.cc +++ b/modules/video_coding/codecs/vp8/libvpx_vp8_encoder.cc @@ -1383,6 +1383,8 @@ LibvpxVp8Encoder::PrepareBuffers(rtc::scoped_refptr buffer) { << VideoFrameBufferTypeToString(buffer_to_scale->type()) << ", the image was unexpectedly converted to " << VideoFrameBufferTypeToString(scaled_buffer->type()) + << " instead of " + << VideoFrameBufferTypeToString(mapped_buffer->type()) << ". Can't encode frame."; return {}; }