Commit 3322d3c7 authored by alokp's avatar alokp Committed by Commit bot

Create and use mojo::Binding on the same thread.

Review URL: https://codereview.chromium.org/1517753002

Cr-Commit-Position: refs/heads/master@{#364448}
parent 5f8fc8a0
...@@ -21,7 +21,6 @@ MojoRendererImpl::MojoRendererImpl( ...@@ -21,7 +21,6 @@ MojoRendererImpl::MojoRendererImpl(
interfaces::RendererPtr remote_renderer) interfaces::RendererPtr remote_renderer)
: task_runner_(task_runner), : task_runner_(task_runner),
remote_renderer_(remote_renderer.Pass()), remote_renderer_(remote_renderer.Pass()),
binding_(this),
weak_factory_(this) { weak_factory_(this) {
DVLOG(1) << __FUNCTION__; DVLOG(1) << __FUNCTION__;
} }
...@@ -67,7 +66,8 @@ void MojoRendererImpl::Initialize( ...@@ -67,7 +66,8 @@ void MojoRendererImpl::Initialize(
new MojoDemuxerStreamImpl(video, GetProxy(&video_stream)); new MojoDemuxerStreamImpl(video, GetProxy(&video_stream));
interfaces::RendererClientPtr client_ptr; interfaces::RendererClientPtr client_ptr;
binding_.Bind(GetProxy(&client_ptr)); binding_.reset(
new mojo::Binding<RendererClient>(this, GetProxy(&client_ptr)));
remote_renderer_->Initialize( remote_renderer_->Initialize(
client_ptr.Pass(), audio_stream.Pass(), video_stream.Pass(), client_ptr.Pass(), audio_stream.Pass(), video_stream.Pass(),
BindToCurrentLoop(base::Bind(&MojoRendererImpl::OnInitialized, BindToCurrentLoop(base::Bind(&MojoRendererImpl::OnInitialized,
......
...@@ -66,7 +66,7 @@ class MojoRendererImpl : public Renderer, public interfaces::RendererClient { ...@@ -66,7 +66,7 @@ class MojoRendererImpl : public Renderer, public interfaces::RendererClient {
DemuxerStreamProvider* demuxer_stream_provider_; DemuxerStreamProvider* demuxer_stream_provider_;
interfaces::RendererPtr remote_renderer_; interfaces::RendererPtr remote_renderer_;
mojo::Binding<RendererClient> binding_; scoped_ptr<mojo::Binding<RendererClient>> binding_;
// Callbacks passed to Initialize() that we forward messages from // Callbacks passed to Initialize() that we forward messages from
// |remote_renderer_| through. // |remote_renderer_| through.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment