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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/media/router/cast/CastMediaRouteProvider.java

Issue 1361763006: [Presentation API, Android] Plumbing SendBinaryMessage() from C++ MediaRouterAndroid to java side. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 5 years, 2 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/android/java/src/org/chromium/chrome/browser/media/router/cast/CastMediaRouteProvider.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/media/router/cast/CastMediaRouteProvider.java b/chrome/android/java/src/org/chromium/chrome/browser/media/router/cast/CastMediaRouteProvider.java
index b3cd9a425efeeb881d6d49569e6f881315d1ea9e..6acdd56d401f8f5891541c2c2be226718858a021 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/media/router/cast/CastMediaRouteProvider.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/media/router/cast/CastMediaRouteProvider.java
@@ -292,6 +292,15 @@ public class CastMediaRouteProvider
route.sendStringMessage(message, nativeCallbackId);
}
+ @Override
+ public void sendBinaryMessage(String routeId, byte[] data, int nativeCallbackId) {
+ // TODO(crbug.com/524128): Cast API does not support sending binary message
+ // to receiver application. Binary data may be converted to String and send as
+ // an app_message within it's own message namespace, using the string version.
+ // Sending failure in the result callback for now.
+ mManager.onMessageSentResult(false, nativeCallbackId);
+ }
+
private CastMediaRouteProvider(
Context applicationContext, MediaRouter androidMediaRouter, MediaRouteManager manager) {
mApplicationContext = applicationContext;

Powered by Google App Engine
This is Rietveld 408576698