Move ownership of audio/video sinks to MojoRendererService.

media/mojo/services/mojo_renderer_service.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef MEDIA_MOJO_SERVICES_MOJO_RENDERER_SERVICE_H_
 #define MEDIA_MOJO_SERVICES_MOJO_RENDERER_SERVICE_H_
 
 #include <stdint.h>
 #include <memory>
 
 #include "base/callback.h"
 #include "base/compiler_specific.h"
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/weak_ptr.h"
 #include "base/timer/timer.h"
 #include "media/base/buffering_state.h"
 #include "media/base/demuxer_stream_provider.h"
 #include "media/base/pipeline_status.h"
 #include "media/base/renderer_client.h"
 #include "media/mojo/interfaces/renderer.mojom.h"
 #include "media/mojo/services/media_mojo_export.h"
 #include "mojo/public/cpp/bindings/strong_binding.h"
 
 namespace media {
 
+class AudioRendererSink;
 class DemuxerStreamProviderShim;
 class MediaKeys;
 class MojoCdmServiceContext;
 class Renderer;
+class VideoRendererSink;
 
 // A mojom::Renderer implementation that use a media::Renderer to render
 // media streams.
 class MEDIA_MOJO_EXPORT MojoRendererService
     : NON_EXPORTED_BASE(public mojom::Renderer),
       public RendererClient {
  public:
   // |mojo_cdm_service_context| can be used to find the CDM to support
   // encrypted media. If null, encrypted media is not supported.
   MojoRendererService(
       base::WeakPtr<MojoCdmServiceContext> mojo_cdm_service_context,
+      scoped_refptr<AudioRendererSink> audio_sink,
+      std::unique_ptr<VideoRendererSink> video_sink,
       std::unique_ptr<media::Renderer> renderer,
       mojo::InterfaceRequest<mojom::Renderer> request);
   ~MojoRendererService() final;
 
   // mojom::Renderer implementation.
   void Initialize(mojom::RendererClientPtr client,
                   mojom::DemuxerStreamPtr audio,
                   mojom::DemuxerStreamPtr video,
                   const base::Optional<GURL>& url,
                   const InitializeCallback& callback) final;
@@ skipping 55 matching lines @@
 
   base::RepeatingTimer time_update_timer_;
   base::TimeDelta last_media_time_;
 
   mojom::RendererClientPtr client_;
 
   // Hold a reference to the CDM set on the |renderer_| so that the CDM won't be
   // destructed while the |renderer_| is still using it.
   scoped_refptr<MediaKeys> cdm_;
 
+  // Audio and Video sinks.
+  // May be null if underlying |renderer_| does not use them.
+  scoped_refptr<AudioRendererSink> audio_sink_;
+  std::unique_ptr<VideoRendererSink> video_sink_;
+
   // Note: Destroy |renderer_| first to avoid access violation into other
-  // members, e.g. |stream_provider_| and |cdm_|.
+  // members, e.g. |stream_provider_|, |cdm_|, |audio_sink_|, and
+  // |video_sink_|.
   // Must use "media::" because "Renderer" is ambiguous.
   std::unique_ptr<media::Renderer> renderer_;
 
   base::WeakPtr<MojoRendererService> weak_this_;
   base::WeakPtrFactory<MojoRendererService> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(MojoRendererService);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_MOJO_SERVICES_MOJO_RENDERER_SERVICE_H_