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

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

Issue 2016053003: Add audio_device_id in mojo media ServiceFactory CreateRenderer api. (Closed) Base URL: https://chromium.googlesource.com/chromium/src@master
Patch Set: Fix comments. Created 4 years, 6 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_TEST_MOJO_MEDIA_CLIENT_H_ 5 #ifndef MEDIA_MOJO_SERVICES_TEST_MOJO_MEDIA_CLIENT_H_
6 #define MEDIA_MOJO_SERVICES_TEST_MOJO_MEDIA_CLIENT_H_ 6 #define MEDIA_MOJO_SERVICES_TEST_MOJO_MEDIA_CLIENT_H_
7 7
8 #include <memory> 8 #include <memory>
9 9
10 #include "base/macros.h" 10 #include "base/macros.h"
11 #include "base/memory/ref_counted.h" 11 #include "base/memory/ref_counted.h"
12 #include "media/audio/audio_manager.h" 12 #include "media/audio/audio_manager.h"
13 #include "media/mojo/services/mojo_media_client.h" 13 #include "media/mojo/services/mojo_media_client.h"
14 14
15 namespace base {
16 class SingleThreadTaskRunner;
17 }
18
15 namespace media { 19 namespace media {
16 20
17 class AudioHardwareConfig; 21 class AudioHardwareConfig;
18 class AudioRendererSink; 22 class AudioRendererSink;
23 class MediaLog;
24 class RendererFactory;
19 class VideoRendererSink; 25 class VideoRendererSink;
20 26
21 // Default MojoMediaClient for MojoMediaApplication. 27 // Default MojoMediaClient for MojoMediaApplication.
22 class TestMojoMediaClient : public MojoMediaClient { 28 class TestMojoMediaClient : public MojoMediaClient {
23 public: 29 public:
24 TestMojoMediaClient(); 30 TestMojoMediaClient();
25 ~TestMojoMediaClient() final; 31 ~TestMojoMediaClient() final;
26 32
27 // MojoMediaClient implementation. 33 // MojoMediaClient implementation.
28 void Initialize() final; 34 void Initialize() final;
29 void WillQuit() final; 35 void WillQuit() final;
30 std::unique_ptr<RendererFactory> CreateRendererFactory( 36 std::unique_ptr<Renderer> CreateRenderer(
31 const scoped_refptr<MediaLog>& media_log) final; 37 const scoped_refptr<base::SingleThreadTaskRunner>& media_task_runner,
32 AudioRendererSink* CreateAudioRendererSink() final; 38 const scoped_refptr<MediaLog>& media_log,
33 VideoRendererSink* CreateVideoRendererSink( 39 const std::string& audio_device_id) final;
34 const scoped_refptr<base::SingleThreadTaskRunner>& task_runner) final;
35 std::unique_ptr<CdmFactory> CreateCdmFactory( 40 std::unique_ptr<CdmFactory> CreateCdmFactory(
36 shell::mojom::InterfaceProvider* /* interface_provider */) final; 41 shell::mojom::InterfaceProvider* /* interface_provider */) final;
37 42
38 private: 43 private:
44 RendererFactory* CreateRendererFactory(
45 const scoped_refptr<MediaLog>& media_log);
46 AudioRendererSink* CreateAudioRendererSink();
47 VideoRendererSink* CreateVideoRendererSink(
xhwang 2016/06/02 06:09:29 nit: s/Create/Get/ for these 3 functions which mak
tianyuwang1 2016/06/02 21:50:53 Done.
48 const scoped_refptr<base::SingleThreadTaskRunner>& task_runner);
49
39 ScopedAudioManagerPtr audio_manager_; 50 ScopedAudioManagerPtr audio_manager_;
51 std::unique_ptr<RendererFactory> renderer_factory_;
40 scoped_refptr<AudioRendererSink> audio_renderer_sink_; 52 scoped_refptr<AudioRendererSink> audio_renderer_sink_;
41 std::unique_ptr<VideoRendererSink> video_renderer_sink_; 53 std::unique_ptr<VideoRendererSink> video_renderer_sink_;
42 54
43 DISALLOW_COPY_AND_ASSIGN(TestMojoMediaClient); 55 DISALLOW_COPY_AND_ASSIGN(TestMojoMediaClient);
44 }; 56 };
45 57
46 } // namespace media 58 } // namespace media
47 59
48 #endif // MEDIA_MOJO_SERVICES_TEST_MOJO_MEDIA_CLIENT_H_ 60 #endif // MEDIA_MOJO_SERVICES_TEST_MOJO_MEDIA_CLIENT_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698