Added initial set of logging for AudioOutputController.

content/browser/renderer_host/media/audio_output_delegate_impl.h

Index: content/browser/renderer_host/media/audio_output_delegate_impl.h
diff --git a/content/browser/renderer_host/media/audio_output_delegate_impl.h b/content/browser/renderer_host/media/audio_output_delegate_impl.h
index 9a2c89aef7d45a985db4b54af8c8ec1340696fca..d4b9435f2068150e8859c0956bf404a8b4dd587d 100644
--- a/content/browser/renderer_host/media/audio_output_delegate_impl.h
+++ b/content/browser/renderer_host/media/audio_output_delegate_impl.h
@@ -71,6 +71,7 @@ class CONTENT_EXPORT AudioOutputDelegateImpl
 
   void SendCreatedNotification();
   void OnError();
+  static void DoLog(int stream_id, const std::string& msg);

[liberato (no reviews please), 2017/05/30 16:53:34]

please add a comment about why this needs to be static.  it's a little subtle.

[ossu-chromium, 2017/05/31 11:16:27]

I've looked at this again and I think it would be even better if I just folded
this into OnLog as a lambda instead. There's no reason this function needs to be
in AudioOutputDelegateImpl nor publicly visible.

   void UpdatePlayingState(bool playing);
   media::AudioOutputController* GetControllerForTesting() const;