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

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: override in media_router_android 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..6fd99e080e15a53e3a89959d3bece0011d91cf7d 100644
--- a/chrome/browser/media/router/media_router_mojo_impl_unittest.cc
+++ b/chrome/browser/media/router/media_router_mojo_impl_unittest.cc
@@ -48,6 +48,7 @@ const char kSinkName[] = "sinkName";
const char kPresentationId[] = "presentationId";
const char kOrigin[] = "http://origin/";
const int kTabId = 123;
+const uint8_t kBinaryMessage[] = {0x01, 0x02, 0x03, 0x04};
Kevin M 2015/07/22 18:31:50 uint8
USE s.singapati at gmail.com 2015/07/23 19:52:53 kBinaryMessage is not used now.
bool ArePresentationSessionMessagesEqual(
const content::PresentationSessionMessage* expected,
@@ -390,6 +391,27 @@ TEST_F(MediaRouterMojoImplTest, SendRouteMessage) {
ProcessEventLoop();
}
+TEST_F(MediaRouterMojoImplTest, SendRouteBinaryMessage) {
+ std::vector<uint8_t> in_data;
imcheng 2015/07/22 20:51:31 nit: you can pass in the data range during constru
USE s.singapati at gmail.com 2015/07/23 19:52:53 Done. Filling the array during construction.
+ in_data.assign(kBinaryMessage, kBinaryMessage + sizeof(kBinaryMessage));
Kevin M 2015/07/22 18:31:50 Use arraysize() instead of sizeof()
+
+ EXPECT_CALL(mock_media_route_provider_,
+ SendRouteBinaryMessageInternal(mojo::String(kRouteId),
+ std::vector<uint8_t>(in_data), _))
+ .WillOnce(Invoke([](
+ const MediaRoute::Id& route_id, const std::vector<uint8_t>& data,
+ const interfaces::MediaRouteProvider::SendRouteMessageCallback& cb) {
+ cb.Run(true);
+ }));
+
+ SendMessageCallbackHandler handler;
+ EXPECT_CALL(handler, Invoke(true));
+ router()->SendRouteBinaryMessage(
+ kRouteId, in_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