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

Unified Diff: chrome/browser/media/router/media_source_helper_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_helper_unittest.cc
diff --git a/chrome/browser/media/router/media_source_helper_unittest.cc b/chrome/browser/media/router/media_source_helper_unittest.cc
index c556f6cc1297a96a410cd444767e7a3f73362e36..1b708fd3afc8a39d6957566efa645d792f691370 100644
--- a/chrome/browser/media/router/media_source_helper_unittest.cc
+++ b/chrome/browser/media/router/media_source_helper_unittest.cc
@@ -45,5 +45,24 @@ TEST(MediaSourcesTest, IsValidMediaSource) {
MediaSourceForPresentationUrl(GURL("totally not a url"))));
}
+TEST(MediaSourcesTest, CanConnectToMediaSource) {
+ EXPECT_TRUE(CanConnectToMediaSource(
+ MediaSource(GURL("https://google.com/cast#__castAppId__=233637DE"))));
+ // false scheme
+ EXPECT_FALSE(CanConnectToMediaSource(
+ MediaSource(GURL("http://google.com/cast#__castAppId__=233637DE"))));
+ // false domain
+ EXPECT_FALSE(CanConnectToMediaSource(
+ MediaSource(GURL("https://google2.com/cast#__castAppId__=233637DE"))));
+ // empty path
+ EXPECT_FALSE(
+ CanConnectToMediaSource(MediaSource(GURL("https://www.google.com"))));
+ // false path
+ EXPECT_FALSE(CanConnectToMediaSource(
+ MediaSource(GURL("https://www.google.com/path"))));
+
+ EXPECT_FALSE(CanConnectToMediaSource(MediaSource(GURL(""))));
+}
+
} // namespace media_router
« no previous file with comments | « chrome/browser/media/router/media_source_helper.cc ('k') | chrome/browser/media/router/media_source_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698