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

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

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 #ifndef MEDIA_MOJO_SERVICES_MOJO_RENDERER_IMPL_H_ 5 #ifndef MEDIA_MOJO_SERVICES_MOJO_RENDERER_IMPL_H_
6 #define MEDIA_MOJO_SERVICES_MOJO_RENDERER_IMPL_H_ 6 #define MEDIA_MOJO_SERVICES_MOJO_RENDERER_IMPL_H_
7 7
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "base/memory/weak_ptr.h" 9 #include "base/memory/weak_ptr.h"
10 #include "media/base/renderer.h" 10 #include "media/base/renderer.h"
11 #include "media/mojo/interfaces/media_renderer.mojom.h" 11 #include "media/mojo/interfaces/media_renderer.mojom.h"
12 12
13 namespace base { 13 namespace base {
14 class SingleThreadTaskRunner; 14 class SingleThreadTaskRunner;
15 } 15 }
16 16
17 namespace mojo { 17 namespace media {
18 class ServiceProvider;
19 }
20 18
21 namespace media {
22 class DemuxerStreamProvider; 19 class DemuxerStreamProvider;
23 20
24 // A media::Renderer that proxies to a mojo::MediaRenderer. That 21 // A media::Renderer that proxies to a mojo::MediaRenderer. That
25 // mojo::MediaRenderer proxies back to the MojoRendererImpl via the 22 // mojo::MediaRenderer proxies back to the MojoRendererImpl via the
26 // mojo::MediaRendererClient interface. 23 // mojo::MediaRendererClient interface.
27 // 24 //
28 // MojoRendererImpl implements media::Renderer for use as either an audio 25 // MojoRendererImpl implements media::Renderer for use as either an audio
29 // or video renderer. 26 // or video renderer.
30 class MojoRendererImpl : public Renderer, public mojo::MediaRendererClient { 27 class MojoRendererImpl : public Renderer, public mojo::MediaRendererClient {
31 public: 28 public:
32 // |task_runner| is the TaskRunner on which all methods are invoked. 29 // |task_runner| is the TaskRunner on which all methods are invoked.
33 // |media_renderer_provider| is a ServiceProvider from a connected
34 // Application that is hosting a mojo::MediaRenderer.
35 MojoRendererImpl( 30 MojoRendererImpl(
36 const scoped_refptr<base::SingleThreadTaskRunner>& task_runner, 31 const scoped_refptr<base::SingleThreadTaskRunner>& task_runner,
37 mojo::ServiceProvider* media_renderer_provider); 32 mojo::MediaRendererPtr remote_media_renderer);
38 ~MojoRendererImpl() override; 33 ~MojoRendererImpl() override;
39 34
40 // Renderer implementation. 35 // Renderer implementation.
41 void Initialize(DemuxerStreamProvider* demuxer_stream_provider, 36 void Initialize(DemuxerStreamProvider* demuxer_stream_provider,
42 const base::Closure& init_cb, 37 const base::Closure& init_cb,
43 const StatisticsCB& statistics_cb, 38 const StatisticsCB& statistics_cb,
44 const BufferingStateCB& buffering_state_cb, 39 const BufferingStateCB& buffering_state_cb,
45 const PaintCB& paint_cb, 40 const PaintCB& paint_cb,
46 const base::Closure& ended_cb, 41 const base::Closure& ended_cb,
47 const PipelineStatusCB& error_cb) override; 42 const PipelineStatusCB& error_cb) override;
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
86 base::TimeDelta max_time_; 81 base::TimeDelta max_time_;
87 82
88 base::WeakPtrFactory<MojoRendererImpl> weak_factory_; 83 base::WeakPtrFactory<MojoRendererImpl> weak_factory_;
89 84
90 DISALLOW_COPY_AND_ASSIGN(MojoRendererImpl); 85 DISALLOW_COPY_AND_ASSIGN(MojoRendererImpl);
91 }; 86 };
92 87
93 } // namespace media 88 } // namespace media
94 89
95 #endif // MEDIA_MOJO_SERVICES_MOJO_RENDERER_IMPL_H_ 90 #endif // MEDIA_MOJO_SERVICES_MOJO_RENDERER_IMPL_H_
OLDNEW
« no previous file with comments | « media/mojo/services/media_renderer_apptest.cc ('k') | media/mojo/services/mojo_renderer_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698