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

Unified Diff: chrome/browser/ui/webui/media_router/media_router_ui_unittest.cc

Issue 2547703002: [Media Router] Handle multiple Presentation URLs when creating routes (Closed)
Patch Set: fix unittests 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/ui/webui/media_router/media_router_ui_unittest.cc
diff --git a/chrome/browser/ui/webui/media_router/media_router_ui_unittest.cc b/chrome/browser/ui/webui/media_router/media_router_ui_unittest.cc
index 67e41c687f6cda753faee488f5e9ba55d17bffb0..9c2d3f8b8f1b0c7c22b0b9f352948d29cf12321b 100644
--- a/chrome/browser/ui/webui/media_router/media_router_ui_unittest.cc
+++ b/chrome/browser/ui/webui/media_router/media_router_ui_unittest.cc
@@ -487,7 +487,7 @@ TEST_F(MediaRouterUITest, NotFoundErrorOnCloseWithNoSinks) {
"No screens found.");
PresentationRequestCallbacks request_callbacks(expected_error);
create_session_request_.reset(new CreatePresentationConnectionRequest(
- RenderFrameHostId(0, 0), GURL("http://google.com/presentation"),
+ RenderFrameHostId(0, 0), {GURL("http://google.com/presentation")},
mark a. foltz 2016/12/02 20:59:09 It would be good to update test cases to take mult
zhaobin 2016/12/02 22:50:57 Done.
GURL("http://google.com"),
base::Bind(&PresentationRequestCallbacks::Success,
base::Unretained(&request_callbacks)),
@@ -506,7 +506,7 @@ TEST_F(MediaRouterUITest, NotFoundErrorOnCloseWithNoCompatibleSinks) {
PresentationRequestCallbacks request_callbacks(expected_error);
GURL presentation_url("http://google.com/presentation");
create_session_request_.reset(new CreatePresentationConnectionRequest(
- RenderFrameHostId(0, 0), presentation_url, GURL("http://google.com"),
+ RenderFrameHostId(0, 0), {presentation_url}, GURL("http://google.com"),
base::Bind(&PresentationRequestCallbacks::Success,
base::Unretained(&request_callbacks)),
base::Bind(&PresentationRequestCallbacks::Error,
@@ -536,7 +536,7 @@ TEST_F(MediaRouterUITest, AbortErrorOnClose) {
PresentationRequestCallbacks request_callbacks(expected_error);
GURL presentation_url("http://google.com/presentation");
create_session_request_.reset(new CreatePresentationConnectionRequest(
- RenderFrameHostId(0, 0), presentation_url, GURL("http://google.com"),
+ RenderFrameHostId(0, 0), {presentation_url}, GURL("http://google.com"),
base::Bind(&PresentationRequestCallbacks::Success,
base::Unretained(&request_callbacks)),
base::Bind(&PresentationRequestCallbacks::Error,

Powered by Google App Engine
This is Rietveld 408576698