commit 6ba7feb302b478d13a586e3a35b6ecb28dd9977e
author Philipp Hancke <phancke@microsoft.com> Wed Sep 13 07:54:49 2023
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Sep 13 15:54:36 2023
tree d7a74f70a20091271c228cb5b51999d7c1dbaeef
parent 31718d7ce2301eba29babc15df87f2107f79c6ac

Make video encoder reconfiguration logging more verbose

logging the configuration, in particular the content type which
together with RTP configuration information like the ssrcs helps differentiating between encoders.

BUG=None

Change-Id: I1b4b2ec2bffea338cc73c3a9c6a3f775d8f1c26b
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/319560
Reviewed-by: Henrik Boström <hbos@webrtc.org>
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Commit-Queue: Philipp Hancke <phancke@microsoft.com>
Cr-Commit-Position: refs/heads/main@{#40744}

video/config/video_encoder_config.cc

diff --git a/video/config/video_encoder_config.cc b/video/config/video_encoder_config.cc
index 5eec7b5..84442ae 100644
--- a/video/config/video_encoder_config.cc
+++ b/video/config/video_encoder_config.cc
@@ -68,8 +68,7 @@
 std::string VideoEncoderConfig::ToString() const {
   char buf[1024];
   rtc::SimpleStringBuilder ss(buf);
-  ss << "{codec_type: ";
-  ss << CodecTypeToPayloadString(codec_type);
+  ss << "{codec_type: " << CodecTypeToPayloadString(codec_type);
   ss << ", content_type: ";
   switch (content_type) {
     case ContentType::kRealtimeVideo:

video/video_send_stream.cc

diff --git a/video/video_send_stream.cc b/video/video_send_stream.cc
index a01c100..571eb62 100644
--- a/video/video_send_stream.cc
+++ b/video/video_send_stream.cc
@@ -282,6 +282,8 @@
                                               SetParametersCallback callback) {
   RTC_DCHECK_RUN_ON(&thread_checker_);
   RTC_DCHECK_EQ(content_type_, config.content_type);
+  RTC_LOG(LS_VERBOSE) << "Encoder config: " << config.ToString()
+                      << " VideoSendStream config: " << config_.ToString();
   video_stream_encoder_->ConfigureEncoder(
       std::move(config),
       config_.rtp.max_packet_size - CalculateMaxHeaderSize(config_.rtp),