Forward output glitch information from stream WebRTC log

media/audio/audio_logging.h

Index: media/audio/audio_logging.h
diff --git a/media/audio/audio_logging.h b/media/audio/audio_logging.h
index 5b1848ea7a6c0c4e2931152fee868c7e9d8c289a..622e9c89d212a864355b6f428ebecb1f80f7c48a 100644
--- a/media/audio/audio_logging.h
+++ b/media/audio/audio_logging.h
@@ -10,6 +10,7 @@
 
 
 namespace media {
+
 class AudioParameters;
 
 // AudioLog logs state information about an active audio component.  Each method
@@ -48,6 +49,11 @@ class AudioLog {
   // new audio output device.
   virtual void OnSwitchOutputDevice(int component_id,
                                     const std::string& device_id) = 0;
+
+  // Stats reporting.
+  virtual void OnStatistics(int component_id,

[DaleCurtis, 2016/05/19 20:34:37]

It seems this should just be OnStatistics(component_id, string)? Otherwise
you're enforcing all statistics to be of integer type.

[Henrik Grunell, 2016/05/23 17:13:54]

I did consider just using a string (and call it OnMessage() or so). My idea with
string + value was to allow media-internals to track the stats in the future and
int would do for most cases. We could of course change that if we want to add
stats to media-internals later.

If we only pass a string we're not bound to use the AudioLog, so for input we
can just use the existing AudioInputCOntroller::EventHandler::OnLog() which
would simplify a bit there.

I'm fine with either, actually I lean towards just a string and change the name
to AudioLog::OnMessage(), for using existing path on input and avoid adding
extra code.

I've changed to string only.

+                            const std::string& name,
+                            int value) = 0;
 };
 
 // AudioLogFactory dispenses AudioLog instances to owning classes for tracking