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

Unified Diff: chrome/browser/media/router/media_source_unittest.cc

Issue 2547703002: [Media Router] Handle multiple Presentation URLs when creating routes (Closed)
Patch Set: rebase Created 4 years 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: chrome/browser/media/router/media_source_unittest.cc
diff --git a/chrome/browser/media/router/media_source_unittest.cc b/chrome/browser/media/router/media_source_unittest.cc
index 470b5728b6c24f4572038c45b0882d08de00656e..a4e92148b06b1095f353021629b2bbde50095126 100644
--- a/chrome/browser/media/router/media_source_unittest.cc
+++ b/chrome/browser/media/router/media_source_unittest.cc
@@ -11,6 +11,14 @@ namespace media_router {
TEST(MediaSourceTest, Constructor) {
MediaSource source1("urn:x-com.google.cast:application:DEADBEEF");
EXPECT_EQ("urn:x-com.google.cast:application:DEADBEEF", source1.id());
+ EXPECT_EQ(GURL(""), source1.url());
+}
+
+TEST(MediaSourceTest, ConstructorWithGURL) {
+ GURL test_url = GURL("http://google.com");
+ MediaSource source1(test_url);
+ EXPECT_EQ(test_url.spec(), source1.id());
+ EXPECT_EQ(test_url, source1.url());
}
} // namespace media_router
« no previous file with comments | « chrome/browser/media/router/media_source_helper_unittest.cc ('k') | chrome/browser/media/router/presentation_request.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698