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

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

Issue 2737413003: [Presentation API] Remove references to presentation sessions. (Closed)
Patch Set: Update PresentationServiceDelegateImpl unittest Created 3 years, 9 months 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 6c373b9fb3b43c96dfee422472297bccaf2676fa..1597df2e69a37c8ce10eee67fe3683f8c60d7471 100644
--- a/chrome/browser/media/router/create_presentation_connection_request_unittest.cc
+++ b/chrome/browser/media/router/create_presentation_connection_request_unittest.cc
@@ -32,8 +32,8 @@ class CreatePresentationConnectionRequestTest : public ::testing::Test {
~CreatePresentationConnectionRequestTest() override {}
- void OnSuccess(const content::PresentationSessionInfo& expected_info,
- const content::PresentationSessionInfo& actual_info,
+ void OnSuccess(const content::PresentationInfo& expected_info,
+ const content::PresentationInfo& actual_info,
const MediaRoute& route) {
cb_invoked_ = true;
EXPECT_EQ(expected_info.presentation_url, actual_info.presentation_url);
@@ -48,7 +48,7 @@ class CreatePresentationConnectionRequestTest : public ::testing::Test {
EXPECT_EQ(expected_error.message, actual_error.message);
}
- void FailOnSuccess(const content::PresentationSessionInfo& info,
+ void FailOnSuccess(const content::PresentationInfo& info,
const MediaRoute& route) {
FAIL() << "Success callback should not have been called.";
}
@@ -82,12 +82,12 @@ TEST_F(CreatePresentationConnectionRequestTest, Getters) {
}
TEST_F(CreatePresentationConnectionRequestTest, SuccessCallback) {
- content::PresentationSessionInfo session_info(presentation_url_,
- kPresentationId);
+ content::PresentationInfo presentation_info(presentation_url_,
+ kPresentationId);
CreatePresentationConnectionRequest request(
render_frame_host_id_, {presentation_url_}, url::Origin(GURL(kFrameUrl)),
base::Bind(&CreatePresentationConnectionRequestTest::OnSuccess,
- base::Unretained(this), session_info),
+ base::Unretained(this), presentation_info),
base::Bind(&CreatePresentationConnectionRequestTest::FailOnError,
base::Unretained(this)));
MediaRoute route(kRouteId, MediaSourceForTab(1), "sinkId", "Description",
@@ -98,7 +98,7 @@ TEST_F(CreatePresentationConnectionRequestTest, SuccessCallback) {
TEST_F(CreatePresentationConnectionRequestTest, ErrorCallback) {
content::PresentationError error(
- content::PRESENTATION_ERROR_SESSION_REQUEST_CANCELLED,
+ content::PRESENTATION_ERROR_PRESENTATION_REQUEST_CANCELLED,
"This is an error message");
CreatePresentationConnectionRequest request(
render_frame_host_id_, presentation_urls_, url::Origin(GURL(kFrameUrl)),
« 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