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

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

Issue 2379703002: [Presentation API] (alternative) 1-UA: send message between controller and receiver page (Closed)
Patch Set: merge with master Created 4 years, 1 month 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/create_presentation_connection_request_unittest.cc
diff --git a/chrome/browser/media/router/create_presentation_connection_request_unittest.cc b/chrome/browser/media/router/create_presentation_connection_request_unittest.cc
index 6b48c863c4165cd6606cc8b98d004de2ce6dcd63..6d9fdf30f68072ea62a98b81c97f2cf7154b5aad 100644
--- a/chrome/browser/media/router/create_presentation_connection_request_unittest.cc
+++ b/chrome/browser/media/router/create_presentation_connection_request_unittest.cc
@@ -32,7 +32,7 @@ class CreatePresentationConnectionRequestTest : public ::testing::Test {
void OnSuccess(const content::PresentationSessionInfo& expected_info,
const content::PresentationSessionInfo& actual_info,
- const MediaRoute::Id& route_id) {
+ const MediaRoute& route) {
cb_invoked_ = true;
EXPECT_EQ(expected_info.presentation_url, actual_info.presentation_url);
EXPECT_EQ(expected_info.presentation_id, actual_info.presentation_id);
@@ -46,7 +46,7 @@ class CreatePresentationConnectionRequestTest : public ::testing::Test {
}
void FailOnSuccess(const content::PresentationSessionInfo& info,
- const MediaRoute::Id& route_id) {
+ const MediaRoute& route) {
FAIL() << "Success callback should not have been called.";
}
@@ -87,7 +87,9 @@ TEST_F(CreatePresentationConnectionRequestTest, SuccessCallback) {
base::Unretained(this), session_info),
base::Bind(&CreatePresentationConnectionRequestTest::FailOnError,
base::Unretained(this)));
- request.InvokeSuccessCallback(kPresentationId, kRouteId);
+ MediaRoute route(kRouteId, MediaSourceForTab(1), "sinkId", "Description",
+ false, "", false);
+ request.InvokeSuccessCallback(kPresentationId, route);
EXPECT_TRUE(cb_invoked_);
}
« no previous file with comments | « chrome/browser/media/router/create_presentation_connection_request.cc ('k') | chrome/browser/media/router/media_route.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698