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

Unified Diff: content/browser/presentation/presentation_service_impl_unittest.cc

Issue 2471573005: [Presentation API] (5th) (1-UA) integrate controller and receiver side for 1-UA messaging (Closed)
Patch Set: merge and refactor Created 3 years, 11 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: content/browser/presentation/presentation_service_impl_unittest.cc
diff --git a/content/browser/presentation/presentation_service_impl_unittest.cc b/content/browser/presentation/presentation_service_impl_unittest.cc
index e0028b71ae41a94645f6dd5197e8a02d0e39d9db..ddc84c370a6a52f3367c7cd2e926a005ff45909d 100644
--- a/content/browser/presentation/presentation_service_impl_unittest.cc
+++ b/content/browser/presentation/presentation_service_impl_unittest.cc
@@ -147,7 +147,7 @@ class MockPresentationServiceDelegate
const content::PresentationConnectionStateChangedCallback&
state_changed_cb));
- void ConnectToOffscreenPresentation(
+ void ConnectToPresentation(
int render_process_id,
int render_frame_id,
const content::PresentationSessionInfo& session,
@@ -543,8 +543,7 @@ TEST_F(PresentationServiceImplTest, SetDefaultPresentationUrls) {
run_loop.Run();
}
-TEST_F(PresentationServiceImplTest,
- ListenForConnectionStateChangeAndChangeState) {
+TEST_F(PresentationServiceImplTest, ListenForConnectionStateChange) {
content::PresentationSessionInfo connection(presentation_url1_,
kPresentationId);
content::PresentationConnectionStateChangedCallback state_changed_cb;
@@ -555,11 +554,7 @@ TEST_F(PresentationServiceImplTest,
EXPECT_CALL(mock_delegate_, ListenForConnectionStateChange(_, _, _, _))
.WillOnce(SaveArg<3>(&state_changed_cb));
- EXPECT_CALL(mock_client_,
- OnConnectionStateChanged(
- SessionInfoEquals(ByRef(presentation_connection)),
- blink::mojom::PresentationConnectionState::CONNECTED));
- service_impl_->ListenForConnectionStateChangeAndChangeState(connection);
+ service_impl_->ListenForConnectionStateChange(connection);
{
base::RunLoop run_loop;
@@ -580,7 +575,7 @@ TEST_F(PresentationServiceImplTest, ListenForConnectionClose) {
content::PresentationConnectionStateChangedCallback state_changed_cb;
EXPECT_CALL(mock_delegate_, ListenForConnectionStateChange(_, _, _, _))
.WillOnce(SaveArg<3>(&state_changed_cb));
- service_impl_->ListenForConnectionStateChangeAndChangeState(connection);
+ service_impl_->ListenForConnectionStateChange(connection);
// Trigger connection close. It should be propagated back up to
// |mock_client_|.

Powered by Google App Engine
This is Rietveld 408576698