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

Unified 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: 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 side-by-side diff with in-line comments
Download patch
Index: media/base/android/media_url_demuxer.cc
diff --git a/media/base/android/media_url_demuxer.cc b/media/base/android/media_url_demuxer.cc
new file mode 100644
index 0000000000000000000000000000000000000000..9e1fbc0dc0404d66c06f28f77ecdb28ab49b0019
--- /dev/null
+++ b/media/base/android/media_url_demuxer.cc
@@ -0,0 +1,65 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "media/base/android/media_url_demuxer.h"
+
+#include "base/memory/ptr_util.h"
DaleCurtis 2016/06/24 22:02:40 What is this for?
tguilbert 2016/06/24 23:21:47 Removed.
+
+namespace media {
+
+MediaUrlDemuxer::MediaUrlDemuxer(const GURL& url) : url_(url) {}
+
+MediaUrlDemuxer::~MediaUrlDemuxer() {}
+
+// Should never be called since DemuxerStreamProvider::Type is URL.
+DemuxerStream* MediaUrlDemuxer::GetStream(DemuxerStream::Type type) {
+ NOTREACHED();
+ return nullptr;
+}
+
+GURL MediaUrlDemuxer::GetUrl() {
+ return url_;
+}
+
+DemuxerStreamProvider::Type MediaUrlDemuxer::type() {
+ return DemuxerStreamProvider::Type::URL;
+}
+
+std::string MediaUrlDemuxer::GetDisplayName() const {
+ return "MediaUrlDemuxer";
+}
+
+void MediaUrlDemuxer::Initialize(DemuxerHost* host,
+ const PipelineStatusCB& status_cb,
+ bool enable_text_tracks) {
+ DVLOG(1) << __FUNCTION__;
+ status_cb.Run(PIPELINE_OK);
+}
+
+void MediaUrlDemuxer::StartWaitingForSeek(base::TimeDelta seek_time) {}
+
+void MediaUrlDemuxer::CancelPendingSeek(base::TimeDelta seek_time) {}
+
+void MediaUrlDemuxer::Seek(base::TimeDelta time,
+ const PipelineStatusCB& status_cb) {
+ status_cb.Run(PIPELINE_OK);
+}
+
+void MediaUrlDemuxer::Stop() {}
+
+base::TimeDelta MediaUrlDemuxer::GetStartTime() const {
+ // TODO(tguilbert): Investigate if we need to fetch information from the
+ // MediaPlayerRender in order to return a sensible value here.
+ return base::TimeDelta();
+}
+base::Time MediaUrlDemuxer::GetTimelineOffset() const {
+ // This seems to be an appropriate value for the moment.
+ return base::Time();
+}
+
+int64_t MediaUrlDemuxer::GetMemoryUsage() const {
+ return 0;
+}
+
+} // namespace media

Powered by Google App Engine
This is Rietveld 408576698