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

Side by Side Diff: media/base/android/media_url_demuxer.cc

Issue 2090343004: Add GetUrl to DemuxerStreamProvider interface (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: addressing 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
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "media/base/android/media_url_demuxer.h"
6
7 #include "base/memory/ptr_util.h"
8
9 namespace media {
10
11 MediaUrlDemuxer::MediaUrlDemuxer(const GURL& url) : url_(url) {}
12
13 MediaUrlDemuxer::~MediaUrlDemuxer() {}
14
15 DemuxerStream* MediaUrlDemuxer::GetStream(DemuxerStream::Type type) {
16 return nullptr;
xhwang 2016/06/24 03:19:47 NOTREACHED()
tguilbert 2016/06/24 21:18:42 Done.
17 }
18
19 GURL MediaUrlDemuxer::GetUrl() {
20 return url_;
21 }
22
23 std::string MediaUrlDemuxer::GetDisplayName() const {
24 return "MediaUrlDemuxer";
25 }
26
27 void MediaUrlDemuxer::Initialize(DemuxerHost* host,
xhwang 2016/06/24 03:19:47 This function and below should be NOTREACHED()?
tguilbert 2016/06/24 21:18:42 They should not. A lot of them are indeed called b
28 const PipelineStatusCB& status_cb,
29 bool enable_text_tracks) {
30 DVLOG(1) << __FUNCTION__;
31 status_cb.Run(PIPELINE_OK);
32 }
33
34 void MediaUrlDemuxer::StartWaitingForSeek(base::TimeDelta seek_time) {}
35
36 void MediaUrlDemuxer::CancelPendingSeek(base::TimeDelta seek_time) {}
37
38 void MediaUrlDemuxer::Seek(base::TimeDelta time,
39 const PipelineStatusCB& status_cb) {
40 status_cb.Run(PIPELINE_OK);
41 }
42
43 void MediaUrlDemuxer::Stop() {}
44
45 base::TimeDelta MediaUrlDemuxer::GetStartTime() const {
46 // TODO(tguilbert): Investigate if we need to fetch information from the
47 // MediaPlayerRender in order to return a sensible value here.
48 return base::TimeDelta();
49 }
50 base::Time MediaUrlDemuxer::GetTimelineOffset() const {
51 // This seems to be an appropriate value for the moment.
52 return base::Time();
53 }
54
55 int64_t MediaUrlDemuxer::GetMemoryUsage() const {
56 return 0;
57 }
58
59 } // namespace media
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698