Add GetUrl to DemuxerStreamProvider interface

media/base/android/media_url_demuxer.cc

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..841877dc6646aeb80d059891b5938182635397a4
--- /dev/null
+++ b/media/base/android/media_url_demuxer.cc
@@ -0,0 +1,59 @@
+// 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"
+
+namespace media {
+
+MediaUrlDemuxer::MediaUrlDemuxer(const GURL& url) : url_(url) {}
+
+MediaUrlDemuxer::~MediaUrlDemuxer() {}
+
+DemuxerStream* MediaUrlDemuxer::GetStream(DemuxerStream::Type type) {
+  return nullptr;

[xhwang, 2016/06/24 03:19:47]

NOTREACHED()

[tguilbert, 2016/06/24 21:18:42]

Done.

+}
+
+GURL MediaUrlDemuxer::GetUrl() {
+  return url_;
+}
+
+std::string MediaUrlDemuxer::GetDisplayName() const {
+  return "MediaUrlDemuxer";
+}
+
+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 by Pipeline, and it is easier
for the moment to NOP them then to refactor Pipeline.

+                                 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