Change of the format of the log output in the level controller

webrtc/modules/audio_processing/level_controller/level_controller.cc

 /*
  *  Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
@@ skipping 138 matching lines @@
     RTC_HISTOGRAM_COUNTS("WebRTC.Audio.LevelControl.AverageGain",
                          average_gain_db, 0, 33, 30);
 
     const int long_term_peak_level_dbfs = static_cast<int>(
         10 * log10(long_term_peak_level * long_term_peak_level + 1e-10f) -
         kdBFSOffset);
 
     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: {"

[the sun, 2016/09/23 12:31:33]

Consider putting this in a ToString() method, like you did for the Config struct
(would use field names then, rather than plain text, btw).

[peah-webrtc, 2016/10/03 10:37:02]

I agree, but in this case not all the reported metrics are stored as fields as
they are handled before being reported. 

I'll land this CL as it is, but I'd be happy to to change this later.

+                 << "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();
   }
 }
 
 LevelController::LevelController()
     : data_dumper_(new ApmDataDumper(instance_count_)),
       gain_applier_(data_dumper_.get()),
       signal_classifier_(data_dumper_.get()) {
   Initialize(AudioProcessing::kSampleRate48kHz);
@@ skipping 83 matching lines @@
      << "enabled: " << (config.enabled ? "true" : "false") << "}";
   return ss.str();
 }
 
 bool LevelController::Validate(
     const AudioProcessing::Config::LevelController& config) {
   return true;
 }
 
 }  // namespace webrtc