Switch stream creation and closing in Chrome audio rendering from IPC to Mojo

content/public/browser/render_frame_host.h

Index: content/public/browser/render_frame_host.h
diff --git a/content/public/browser/render_frame_host.h b/content/public/browser/render_frame_host.h
index 980704cf16b18af25807b656539ea3bc0105cdc3..4122a73b6a191e324bdd9bf625ec250ef90883fd 100644
--- a/content/public/browser/render_frame_host.h
+++ b/content/public/browser/render_frame_host.h
@@ -24,6 +24,7 @@ class Value;
 }
 
 namespace content {
+class AudioOutputImpl;

[nasko, 2016/05/25 20:50:42]

This is a violation of the content API. No Impl classes are exposed outside of
content/.

[rchtara, 2016/05/27 15:24:39]

Done.

 class RenderProcessHost;
 class RenderViewHost;
 class RenderWidgetHostView;
@@ -50,6 +51,10 @@ class CONTENT_EXPORT RenderFrameHost : public IPC::Listener,
 
   ~RenderFrameHost() override {}
 
+  virtual AudioOutputImpl* GetAudioOutputImpl() = 0;

[nasko, 2016/05/25 20:50:42]

These two methods are not used anywhere, please remove them from the public API.

[rchtara, 2016/05/27 15:24:39]

Done.

+
+  virtual void SetAudioOutputImpl(AudioOutputImpl* audio_output_impl) = 0;

[nasko, 2016/05/25 20:50:42]

Why do we have a set method on the public interface? You also cannot have Impl
classes exposed outside of content, so this is incorrect.

[rchtara, 2016/05/27 15:24:39]

Done.

+
   // Returns the route id for this frame.
   virtual int GetRoutingID() = 0;