content/browser/presentation/presentation_service_impl_unittest.cc - Issue 1199933006: [MediaRouter] wireup send/receive messages.

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

Issue 1199933006: [MediaRouter] wireup send/receive messages. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master

Patch Set: Update presentation_service_impl_unittest Created 5 years, 6 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

« no previous file with comments | « content/browser/presentation/presentation_service_impl.cc ('k') | content/public/browser/presentation_service_delegate.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

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 076b31f79f81b07b771f2568e50947eae4bd0ddc..7a2bd91ee93878f44adf94978396e8ee35d2c9ce 100644

--- a/content/browser/presentation/presentation_service_impl_unittest.cc

+++ b/content/browser/presentation/presentation_service_impl_unittest.cc

@@ -661,7 +661,7 @@ TEST_F(PresentationServiceImplTest, SendStringMessage) {

base::Unretained(this)));

base::RunLoop run_loop;

- base::Closure send_message_cb;

+ base::Callback<void(bool)> send_message_cb;

PresentationSessionMessage* test_message = nullptr;

EXPECT_CALL(mock_delegate_, SendMessageRawPtr(

_, _, _, _))

@@ -680,7 +680,7 @@ TEST_F(PresentationServiceImplTest, SendStringMessage) {

EXPECT_EQ(message, *(test_message->message.get()));

EXPECT_FALSE(test_message->data);

delete test_message;

- send_message_cb.Run();

+ send_message_cb.Run(true);

SaveQuitClosureAndRunLoop();

}

@@ -704,7 +704,7 @@ TEST_F(PresentationServiceImplTest, SendArrayBuffer) {

base::Unretained(this)));

base::RunLoop run_loop;

- base::Closure send_message_cb;

+ base::Callback<void(bool)> send_message_cb;

PresentationSessionMessage* test_message = nullptr;

EXPECT_CALL(mock_delegate_, SendMessageRawPtr(

_, _, _, _))

@@ -725,7 +725,7 @@ TEST_F(PresentationServiceImplTest, SendArrayBuffer) {

kMaxPresentationSessionMessageSize);

EXPECT_EQ(0, memcmp(buffer, &(*test_message->data.get())[0], sizeof(buffer)));

delete test_message;

- send_message_cb.Run();

+ send_message_cb.Run(true);

SaveQuitClosureAndRunLoop();

}

@@ -752,7 +752,7 @@ TEST_F(PresentationServiceImplTest, SendArrayBufferWithExceedingLimit) {

base::Unretained(this)));

base::RunLoop run_loop;

- base::Closure send_message_cb;

+ base::Callback<void(bool)> send_message_cb;

PresentationSessionMessage* test_message = nullptr;

EXPECT_CALL(mock_delegate_, SendMessageRawPtr(

_, _, _, _))

@@ -763,7 +763,7 @@ TEST_F(PresentationServiceImplTest, SendArrayBufferWithExceedingLimit) {

run_loop.Run();

EXPECT_FALSE(test_message);

- send_message_cb.Run();

+ send_message_cb.Run(true);

SaveQuitClosureAndRunLoop();

}

@@ -785,7 +785,7 @@ TEST_F(PresentationServiceImplTest, SendBlobData) {

base::Unretained(this)));

base::RunLoop run_loop;

- base::Closure send_message_cb;

+ base::Callback<void(bool)> send_message_cb;

PresentationSessionMessage* test_message = nullptr;

EXPECT_CALL(mock_delegate_, SendMessageRawPtr(_, _, _, _))

.WillOnce(DoAll(

@@ -805,7 +805,7 @@ TEST_F(PresentationServiceImplTest, SendBlobData) {

kMaxPresentationSessionMessageSize);

EXPECT_EQ(0, memcmp(buffer, &(*test_message->data.get())[0], sizeof(buffer)));

delete test_message;

- send_message_cb.Run();

+ send_message_cb.Run(true);

SaveQuitClosureAndRunLoop();

}