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

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

Issue 655713003: Standardize usage of virtual/override/final in media/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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 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"
(...skipping 19 matching lines...) Expand all
30 // 30 //
31 // TODO(tim): Only audio is currently supported. http://crbug.com/410451. 31 // TODO(tim): Only audio is currently supported. http://crbug.com/410451.
32 class MojoRendererImpl : public Renderer, public mojo::MediaRendererClient { 32 class MojoRendererImpl : public Renderer, public mojo::MediaRendererClient {
33 public: 33 public:
34 // |task_runner| is the TaskRunner on which all methods are invoked. 34 // |task_runner| is the TaskRunner on which all methods are invoked.
35 // |audio_renderer_provider| is a ServiceProvider from a connected 35 // |audio_renderer_provider| is a ServiceProvider from a connected
36 // Application that is hosting a mojo::MediaRenderer. 36 // Application that is hosting a mojo::MediaRenderer.
37 MojoRendererImpl( 37 MojoRendererImpl(
38 const scoped_refptr<base::SingleThreadTaskRunner>& task_runner, 38 const scoped_refptr<base::SingleThreadTaskRunner>& task_runner,
39 mojo::ServiceProvider* audio_renderer_provider); 39 mojo::ServiceProvider* audio_renderer_provider);
40 virtual ~MojoRendererImpl(); 40 ~MojoRendererImpl() override;
41 41
42 // Renderer implementation. 42 // Renderer implementation.
43 virtual void Initialize(DemuxerStreamProvider* demuxer_stream_provider, 43 void Initialize(DemuxerStreamProvider* demuxer_stream_provider,
44 const base::Closure& init_cb, 44 const base::Closure& init_cb,
45 const StatisticsCB& statistics_cb, 45 const StatisticsCB& statistics_cb,
46 const base::Closure& ended_cb, 46 const base::Closure& ended_cb,
47 const PipelineStatusCB& error_cb, 47 const PipelineStatusCB& error_cb,
48 const BufferingStateCB& buffering_state_cb) override; 48 const BufferingStateCB& buffering_state_cb) override;
49 virtual void Flush(const base::Closure& flush_cb) override; 49 void Flush(const base::Closure& flush_cb) override;
50 virtual void StartPlayingFrom(base::TimeDelta time) override; 50 void StartPlayingFrom(base::TimeDelta time) override;
51 virtual void SetPlaybackRate(float playback_rate) override; 51 void SetPlaybackRate(float playback_rate) override;
52 virtual void SetVolume(float volume) override; 52 void SetVolume(float volume) override;
53 virtual base::TimeDelta GetMediaTime() override; 53 base::TimeDelta GetMediaTime() override;
54 virtual bool HasAudio() override; 54 bool HasAudio() override;
55 virtual bool HasVideo() override; 55 bool HasVideo() override;
56 virtual void SetCdm(MediaKeys* cdm) override; 56 void SetCdm(MediaKeys* cdm) override;
57 57
58 // mojo::MediaRendererClient implementation. 58 // mojo::MediaRendererClient implementation.
59 virtual void OnTimeUpdate(int64_t time_usec, 59 void OnTimeUpdate(int64_t time_usec, int64_t max_time_usec) override;
60 int64_t max_time_usec) override; 60 void OnBufferingStateChange(mojo::BufferingState state) override;
61 virtual void OnBufferingStateChange(mojo::BufferingState state) override; 61 void OnEnded() override;
62 virtual void OnEnded() override; 62 void OnError() override;
63 virtual void OnError() override;
64 63
65 private: 64 private:
66 // Called when |remote_audio_renderer_| has finished initializing. 65 // Called when |remote_audio_renderer_| has finished initializing.
67 void OnInitialized(); 66 void OnInitialized();
68 67
69 // Task runner used to execute pipeline tasks. 68 // Task runner used to execute pipeline tasks.
70 scoped_refptr<base::SingleThreadTaskRunner> task_runner_; 69 scoped_refptr<base::SingleThreadTaskRunner> task_runner_;
71 70
72 DemuxerStreamProvider* demuxer_stream_provider_; 71 DemuxerStreamProvider* demuxer_stream_provider_;
73 mojo::MediaRendererPtr remote_audio_renderer_; 72 mojo::MediaRendererPtr remote_audio_renderer_;
74 73
75 // Callbacks passed to Initialize() that we forward messages from 74 // Callbacks passed to Initialize() that we forward messages from
76 // |remote_audio_renderer_| through. 75 // |remote_audio_renderer_| through.
77 base::Closure init_cb_; 76 base::Closure init_cb_;
78 base::Closure ended_cb_; 77 base::Closure ended_cb_;
79 PipelineStatusCB error_cb_; 78 PipelineStatusCB error_cb_;
80 BufferingStateCB buffering_state_cb_; 79 BufferingStateCB buffering_state_cb_;
81 80
82 base::WeakPtrFactory<MojoRendererImpl> weak_factory_; 81 base::WeakPtrFactory<MojoRendererImpl> weak_factory_;
83 DISALLOW_COPY_AND_ASSIGN(MojoRendererImpl); 82 DISALLOW_COPY_AND_ASSIGN(MojoRendererImpl);
84 }; 83 };
85 84
86 } // namespace media 85 } // namespace media
87 86
88 #endif // MEDIA_MOJO_SERVICES_MOJO_RENDERER_IMPL_H_ 87 #endif // MEDIA_MOJO_SERVICES_MOJO_RENDERER_IMPL_H_
OLDNEW
« no previous file with comments | « media/mojo/services/mojo_demuxer_stream_impl.h ('k') | media/mojo/services/mojo_renderer_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698