Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(538)

Unified Diff: media/mojo/services/mojo_renderer_service.cc

Issue 2640003002: Implement MojoAudioRendererSink and use it in UtilityMojoMediaClient (Closed)
Patch Set: Make sure to stop the audio sink. Also unbind from the audio device thread Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: media/mojo/services/mojo_renderer_service.cc
diff --git a/media/mojo/services/mojo_renderer_service.cc b/media/mojo/services/mojo_renderer_service.cc
index 239e58f3b0174fb1978feb258ffb18293bd4bbe8..7160afd54cb88d3afc621d7d3535288f3228d904 100644
--- a/media/mojo/services/mojo_renderer_service.cc
+++ b/media/mojo/services/mojo_renderer_service.cc
@@ -15,6 +15,7 @@
#include "media/base/renderer.h"
#include "media/base/video_renderer_sink.h"
#include "media/mojo/services/demuxer_stream_provider_shim.h"
+#include "media/mojo/services/mojo_audio_renderer_sink_adapter.h"
#include "media/mojo/services/mojo_cdm_service_context.h"
#include "media/mojo/services/mojo_video_renderer_sink_adapter.h"
@@ -80,6 +81,7 @@ void MojoRendererService::Initialize(
mojom::RendererClientAssociatedPtrInfo client,
mojom::DemuxerStreamPtr audio,
mojom::DemuxerStreamPtr video,
+ mojom::AudioRendererSinkPtr audio_renderer_sink_ptr,
mojom::VideoRendererSinkPtr video_renderer_sink_ptr,
const base::Optional<GURL>& media_url,
const base::Optional<GURL>& first_party_for_cookies,
@@ -90,6 +92,11 @@ void MojoRendererService::Initialize(
client_.Bind(std::move(client));
state_ = STATE_INITIALIZING;
+ if (audio_renderer_sink_ptr) {
+ static_cast<MojoAudioRendererSinkAdapter*>(audio_sink_.get())
+ ->SetClient(std::move(audio_renderer_sink_ptr));
+ }
+
if (video_renderer_sink_ptr) {
static_cast<MojoVideoRendererSinkAdapter*>(video_sink_.get())
->Initialize(std::move(video_renderer_sink_ptr));

Powered by Google App Engine
This is Rietveld 408576698