Fix double close in MojoAudioOutputStream.

content/browser/renderer_host/media/audio_renderer_host.cc

Index: content/browser/renderer_host/media/audio_renderer_host.cc
diff --git a/content/browser/renderer_host/media/audio_renderer_host.cc b/content/browser/renderer_host/media/audio_renderer_host.cc
index 2c79886dcc9177a5998c3410a73fca124e6eb05e..5579d0503969819d483822397bd80751419e3deb 100644
--- a/content/browser/renderer_host/media/audio_renderer_host.cc
+++ b/content/browser/renderer_host/media/audio_renderer_host.cc
@@ -107,7 +107,7 @@ void AudioRendererHost::OnDestruct() const {
 void AudioRendererHost::OnStreamCreated(
     int stream_id,
     base::SharedMemory* shared_memory,
-    base::CancelableSyncSocket* foreign_socket) {
+    std::unique_ptr<base::CancelableSyncSocket> foreign_socket) {

[DaleCurtis, 2017/04/11 18:22:44]

You need something to own this here now or the socket is going to be closed no?

[Max Morin, 2017/04/12 08:16:06]

It's fine to close the socket handle on this end, as the socket is only used by
the remote end.

   DCHECK_CURRENTLY_ON(BrowserThread::IO);
 
   if (!PeerHandle()) {