commit 8a9b0f8e02c7539cf21c3de174dad0f8e2becc06
author peah <peah@webrtc.org> Tue Oct 04 07:06:04 2016
committer Commit bot <commit-bot@chromium.org> Tue Oct 04 07:06:10 2016
tree 947ce9579276dec92d2d6bd2022f7ecbde407481
parent a2ece731bf036a4803a0f3805695cb853bfaea4c

The current format of the log output of the level controller differs from the rest of the log. Furthermore, it is hard to automatically parse. This CL changes the format to be a JSON-type format which does not have those issues.

BUG=webrtc:6408

Review-Url: https://codereview.webrtc.org/2364553003
Cr-Commit-Position: refs/heads/master@{#14489}

webrtc/modules/audio_processing/level_controller/level_controller.cc

diff --git a/webrtc/modules/audio_processing/level_controller/level_controller.cc b/webrtc/modules/audio_processing/level_controller/level_controller.cc
index c0edeb9..247b7d0 100644
--- a/webrtc/modules/audio_processing/level_controller/level_controller.cc
+++ b/webrtc/modules/audio_processing/level_controller/level_controller.cc
@@ -156,23 +156,21 @@
     const int frame_peak_level_dbfs = static_cast<int>(
         10 * log10(frame_peak_level * frame_peak_level + 1e-10f) - kdBFSOffset);
 
-    LOG(LS_INFO) << "Level Controller metrics: " << std::endl
-                 << "Max noise power:              " << max_noise_power_dbfs
-                 << " dBFS" << std::endl
-                 << "Average noise power:          " << average_noise_power_dbfs
-                 << " dBFS" << std::endl
-                 << "Max long term peak level:     " << max_peak_level_dbfs
-                 << " dBFS" << std::endl
+    LOG(LS_INFO) << "Level Controller metrics: {"
+                 << "Max noise power: " << max_noise_power_dbfs << " dBFS, "
+                 << "Average noise power: " << average_noise_power_dbfs
+                 << " dBFS, "
+                 << "Max long term peak level: " << max_peak_level_dbfs
+                 << " dBFS, "
                  << "Average long term peak level: " << average_peak_level_dbfs
-                 << " dBFS" << std::endl
-                 << "Max gain:                     " << max_gain_db << " dB"
-                 << std::endl
-                 << "Average gain:                 " << average_gain_db << " dB"
-                 << std::endl
-                 << "Long term peak level:         "
-                 << long_term_peak_level_dbfs << " dBFS" << std::endl
-                 << "Last frame peak level:        " << frame_peak_level_dbfs
-                 << " dBFS";
+                 << " dBFS, "
+                 << "Max gain: " << max_gain_db << " dB, "
+                 << "Average gain: " << average_gain_db << " dB, "
+                 << "Long term peak level: " << long_term_peak_level_dbfs
+                 << " dBFS, "
+                 << "Last frame peak level: " << frame_peak_level_dbfs
+                 << " dBFS"
+                 << "}";
 
     Reset();
   }