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

Side by Side Diff: media/mojo/services/mojo_renderer_impl.cc

Issue 785643002: Pass mojo::MediaRendererPtr in MojoRendererImpl's constructor. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@media_app
Patch Set: Created 6 years 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "media/mojo/services/mojo_renderer_impl.h" 5 #include "media/mojo/services/mojo_renderer_impl.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/callback_helpers.h" 8 #include "base/callback_helpers.h"
9 #include "base/location.h" 9 #include "base/location.h"
10 #include "base/single_thread_task_runner.h" 10 #include "base/single_thread_task_runner.h"
11 #include "media/base/bind_to_current_loop.h" 11 #include "media/base/bind_to_current_loop.h"
12 #include "media/base/demuxer_stream_provider.h" 12 #include "media/base/demuxer_stream_provider.h"
13 #include "media/mojo/services/mojo_demuxer_stream_impl.h" 13 #include "media/mojo/services/mojo_demuxer_stream_impl.h"
14 #include "mojo/public/cpp/application/connect.h" 14 #include "mojo/public/cpp/application/connect.h"
15 #include "mojo/public/cpp/bindings/interface_impl.h" 15 #include "mojo/public/cpp/bindings/interface_impl.h"
16 #include "mojo/public/interfaces/application/service_provider.mojom.h" 16 #include "mojo/public/interfaces/application/service_provider.mojom.h"
17 17
18 namespace media { 18 namespace media {
19 19
20 MojoRendererImpl::MojoRendererImpl( 20 MojoRendererImpl::MojoRendererImpl(
21 const scoped_refptr<base::SingleThreadTaskRunner>& task_runner, 21 const scoped_refptr<base::SingleThreadTaskRunner>& task_runner,
22 mojo::ServiceProvider* media_renderer_provider) 22 mojo::MediaRendererPtr remote_media_renderer)
23 : task_runner_(task_runner), 23 : task_runner_(task_runner),
24 remote_media_renderer_(remote_media_renderer.Pass()),
24 weak_factory_(this) { 25 weak_factory_(this) {
25 DVLOG(1) << __FUNCTION__; 26 DVLOG(1) << __FUNCTION__;
26 DCHECK(media_renderer_provider); 27 DCHECK(remote_media_renderer_);
27 mojo::ConnectToService(media_renderer_provider, &remote_media_renderer_);
28 remote_media_renderer_.set_client(this); 28 remote_media_renderer_.set_client(this);
29 } 29 }
30 30
31 MojoRendererImpl::~MojoRendererImpl() { 31 MojoRendererImpl::~MojoRendererImpl() {
32 DVLOG(1) << __FUNCTION__; 32 DVLOG(1) << __FUNCTION__;
33 DCHECK(task_runner_->BelongsToCurrentThread()); 33 DCHECK(task_runner_->BelongsToCurrentThread());
34 // Connection to |remote_media_renderer_| will error-out here. 34 // Connection to |remote_media_renderer_| will error-out here.
35 } 35 }
36 36
37 // TODO(xhwang): Support |paint_cb| if needed. 37 // TODO(xhwang): Support |paint_cb| if needed.
(...skipping 163 matching lines...) Expand 10 before | Expand all | Expand 10 after
201 201
202 void MojoRendererImpl::OnInitialized() { 202 void MojoRendererImpl::OnInitialized() {
203 DVLOG(1) << __FUNCTION__; 203 DVLOG(1) << __FUNCTION__;
204 DCHECK(task_runner_->BelongsToCurrentThread()); 204 DCHECK(task_runner_->BelongsToCurrentThread());
205 DCHECK(!init_cb_.is_null()); 205 DCHECK(!init_cb_.is_null());
206 206
207 base::ResetAndReturn(&init_cb_).Run(); 207 base::ResetAndReturn(&init_cb_).Run();
208 } 208 }
209 209
210 } // namespace media 210 } // namespace media
OLDNEW
« no previous file with comments | « media/mojo/services/mojo_renderer_impl.h ('k') | mojo/services/html_viewer/webmediaplayer_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698