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

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

Issue 1245213002: [MediaRouter] Implement send binary message from PSDImpl to MRPManager (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: review fixes Created 5 years, 5 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/media_router_mojo_impl_unittest.cc
diff --git a/chrome/browser/media/router/media_router_mojo_impl_unittest.cc b/chrome/browser/media/router/media_router_mojo_impl_unittest.cc
index c62926952cbab8af3c4e67f9684ffdbee103e2a0..efe7a2e29ac6e5a6a356ee2b61d6d98d17f8bc2c 100644
--- a/chrome/browser/media/router/media_router_mojo_impl_unittest.cc
+++ b/chrome/browser/media/router/media_router_mojo_impl_unittest.cc
@@ -390,6 +390,28 @@ TEST_F(MediaRouterMojoImplTest, SendRouteMessage) {
ProcessEventLoop();
}
+TEST_F(MediaRouterMojoImplTest, SendRouteBinaryMessage) {
+ std::vector<uint8> expected_binary_data(8, 1);
+
+ EXPECT_CALL(
+ mock_media_route_provider_,
+ SendRouteBinaryMessageInternal(
+ mojo::String(kRouteId), std::vector<uint8>(expected_binary_data), _))
+ .WillOnce(Invoke([](
+ const MediaRoute::Id& route_id, const std::vector<uint8>& data,
+ const interfaces::MediaRouteProvider::SendRouteMessageCallback& cb) {
+ cb.Run(true);
Kevin M 2015/07/25 00:24:09 Verify that |data| matches |expected_binary_data|?
USE s.singapati at gmail.com 2015/07/29 11:22:01 Done.
+ }));
+
+ SendMessageCallbackHandler handler;
+ EXPECT_CALL(handler, Invoke(true));
+ router()->SendRouteBinaryMessage(
+ kRouteId, expected_binary_data,
+ base::Bind(&SendMessageCallbackHandler::Invoke,
+ base::Unretained(&handler)));
+ ProcessEventLoop();
+}
+
TEST_F(MediaRouterMojoImplTest, ListenForRouteMessages) {
mojo::Array<interfaces::RouteMessagePtr> mojo_messages(2);
mojo_messages[0] = interfaces::RouteMessage::New();

Powered by Google App Engine
This is Rietveld 408576698