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

Side by Side Diff: media/base/renderer_factory.h

Issue 2769153005: Remove |use_fallback_path_| from WMPI (Closed)
Patch Set: Addressed comment Created 3 years, 7 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
« no previous file with comments | « media/base/media_url_demuxer.cc ('k') | media/base/renderer_factory.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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_BASE_RENDERER_FACTORY_H_ 5 #ifndef MEDIA_BASE_RENDERER_FACTORY_H_
6 #define MEDIA_BASE_RENDERER_FACTORY_H_ 6 #define MEDIA_BASE_RENDERER_FACTORY_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/base/media_export.h" 12 #include "media/base/media_export.h"
13 #include "media/base/media_resource.h"
13 #include "media/base/renderer.h" 14 #include "media/base/renderer.h"
14 #include "media/base/surface_manager.h" 15 #include "media/base/surface_manager.h"
15 16
16 namespace base { 17 namespace base {
17 class SingleThreadTaskRunner; 18 class SingleThreadTaskRunner;
18 class TaskRunner; 19 class TaskRunner;
19 } 20 }
20 21
21 namespace media { 22 namespace media {
22 23
(...skipping 11 matching lines...) Expand all
34 // GetMediaTime() could be called on any thread. 35 // GetMediaTime() could be called on any thread.
35 // The created Renderer can use |audio_renderer_sink| to render audio and 36 // The created Renderer can use |audio_renderer_sink| to render audio and
36 // |video_renderer_sink| to render video. 37 // |video_renderer_sink| to render video.
37 virtual std::unique_ptr<Renderer> CreateRenderer( 38 virtual std::unique_ptr<Renderer> CreateRenderer(
38 const scoped_refptr<base::SingleThreadTaskRunner>& media_task_runner, 39 const scoped_refptr<base::SingleThreadTaskRunner>& media_task_runner,
39 const scoped_refptr<base::TaskRunner>& worker_task_runner, 40 const scoped_refptr<base::TaskRunner>& worker_task_runner,
40 AudioRendererSink* audio_renderer_sink, 41 AudioRendererSink* audio_renderer_sink,
41 VideoRendererSink* video_renderer_sink, 42 VideoRendererSink* video_renderer_sink,
42 const RequestSurfaceCB& request_surface_cb) = 0; 43 const RequestSurfaceCB& request_surface_cb) = 0;
43 44
45 // Returns the MediaResource::Type that should be used with the renderers
46 // created by this factory.
47 // NOTE: Returns Type::STREAM by default.
48 virtual MediaResource::Type GetRequiredMediaResourceType();
49
44 private: 50 private:
45 DISALLOW_COPY_AND_ASSIGN(RendererFactory); 51 DISALLOW_COPY_AND_ASSIGN(RendererFactory);
46 }; 52 };
47 53
48 } // namespace media 54 } // namespace media
49 55
50 #endif // MEDIA_BASE_RENDERER_FACTORY_H_ 56 #endif // MEDIA_BASE_RENDERER_FACTORY_H_
OLDNEW
« no previous file with comments | « media/base/media_url_demuxer.cc ('k') | media/base/renderer_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698