Add GetUrl to DemuxerStreamProvider interface

media/base/android/media_url_demuxer_unittest.cc

+// 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 "base/macros.h"
+#include "base/message_loop/message_loop.h"
+#include "media/base/android/media_url_demuxer.h"
+#include "testing/gmock/include/gmock/gmock.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace media {
+
+class MediaUrlDemuxerTest : public testing::Test {
+ public:
+  MediaUrlDemuxerTest() {}
+
+  void Initialize(const GURL& gurl) {
+    demuxer_.reset(new MediaUrlDemuxer(gurl));
+  }
+
+  void Initialize() { Initialize(GURL(kDefaultUrl)); }
+
+  const std::string kDefaultUrl = "http://example.com/";
+
+  std::unique_ptr<Demuxer> demuxer_;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(MediaUrlDemuxerTest);
+};
+
+TEST_F(MediaUrlDemuxerTest, ReturnsExpectedDemuxerStreamProviderType) {
+  Initialize();
+
+  // Make sure the Demuxer's type is URL.
+  ASSERT_EQ(DemuxerStreamProvider::Type::URL, demuxer_->type());

[DaleCurtis, 2016/06/24 22:02:40]

Prefer expect_eq unless the test truly can't proceed any further if this is
false.

[tguilbert, 2016/06/24 23:21:47]

Done.

+}
+
+TEST_F(MediaUrlDemuxerTest, AcceptsNormalUrl) {
+  Initialize();
+
+  GURL url = demuxer_->GetUrl();
+
+  ASSERT_EQ(kDefaultUrl, url.spec());
+}
+
+TEST_F(MediaUrlDemuxerTest, AcceptsEmptyStrings) {
+  Initialize(GURL());
+

[DaleCurtis, 2016/06/24 22:02:40]

Condense to EXPECT_EQ(GURL::EmptyGURL(), demuxer_->GetUrl()); Will be easier to
understand when an error occurs since it'll directly print the source of "url".
Ditto for above if you want.

[tguilbert, 2016/06/24 23:21:47]

Done.

+  GURL url = demuxer_->GetUrl();
+
+  ASSERT_EQ(GURL::EmptyGURL(), url);
+}
+
+TEST_F(MediaUrlDemuxerTest, VerifyIdempotenceOfGetUrl) {

[DaleCurtis, 2016/06/24 22:02:40]

I don't think this test does anything? :)

[tguilbert, 2016/06/24 23:21:47]

You are right, I didn't update this :)
(When it was GURL*, I wanted to "prove" it was the same pointer and not a new
one each time).

+  Initialize();
+
+  GURL url = demuxer_->GetUrl();
+  GURL url_clone = demuxer_->GetUrl();
+
+  ASSERT_EQ(url, url_clone);
+}
+
+}  // namespace media