[MediaRouter] Implement send binary message from PSDImpl to MRPManager

chrome/browser/media/router/media_router_mojo_impl.cc

Index: chrome/browser/media/router/media_router_mojo_impl.cc
diff --git a/chrome/browser/media/router/media_router_mojo_impl.cc b/chrome/browser/media/router/media_router_mojo_impl.cc
index 976b0eeb70d8ccfb8e3a220ef0bc170bc2c08eff..eb9dfceb4638901b04163168d258315e31d604a6 100644
--- a/chrome/browser/media/router/media_router_mojo_impl.cc
+++ b/chrome/browser/media/router/media_router_mojo_impl.cc
@@ -244,6 +244,17 @@ void MediaRouterMojoImpl::SendRouteMessage(
                         base::Unretained(this), route_id, message, callback));
 }
 
+void MediaRouterMojoImpl::SendRouteBinaryMessage(
+    const MediaRoute::Id& route_id,
+    scoped_ptr<std::vector<uint8>> data,
+    const SendRouteMessageCallback& callback) {
+  DCHECK(thread_checker_.CalledOnValidThread());
+
+  RunOrDefer(base::Bind(&MediaRouterMojoImpl::DoSendSessionBinaryMessage,
+                        base::Unretained(this), route_id, data.get(),

[haibinlu, 2015/07/28 18:57:19]

data.Pass()

[USE s.singapati at gmail.com, 2015/07/28 19:11:46]

Then DoSendSessionBinaryMessage() has to take scoped_ptr<std::vector<uint8>>.
And base::Bind() did not allow to take scoped_ptr<>. I will check it once again.
 

[Kevin M, 2015/07/28 20:03:36]

Use base::Passed(data.Pass()).

If you just pass the bare pointer with .get(), you're passing a bad reference,
because |data| goes out of scope and is deleted as soon as
SendRouteBinaryMessage finishes.

[USE s.singapati at gmail.com, 2015/07/29 11:22:01]

Done.

+                        callback));
+}
+
 void MediaRouterMojoImpl::ListenForRouteMessages(
     const std::vector<MediaRoute::Id>& route_ids,
     const PresentationSessionMessageCallback& message_cb) {
@@ -378,6 +389,17 @@ void MediaRouterMojoImpl::DoSendSessionMessage(
   media_route_provider_->SendRouteMessage(route_id, message, callback);
 }
 
+void MediaRouterMojoImpl::DoSendSessionBinaryMessage(
+    const MediaRoute::Id& route_id,
+    std::vector<uint8>* data,
+    const SendRouteMessageCallback& callback) {
+  DVLOG_WITH_INSTANCE(1) << "SendRouteBinaryMessage " << route_id;
+  mojo::Array<uint8> mojo_array;
+  mojo_array.Swap(data);
+  media_route_provider_->SendRouteBinaryMessage(route_id, mojo_array.Pass(),
+                                                callback);
+}
+
 void MediaRouterMojoImpl::DoListenForRouteMessages(
     const std::vector<MediaRoute::Id>& route_ids,
     const PresentationSessionMessageCallback& message_cb) {