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

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

Issue 1270353003: [PresentationAPI, Android] Add ChromeMediaRouterDialogController.java and bind it with the native M… (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@media-router-jni
Patch Set: Created 5 years, 4 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/ChromeMediaRouterDialogController.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/media/router/ChromeMediaRouter.java b/chrome/android/java/src/org/chromium/chrome/browser/media/router/ChromeMediaRouterDialogController.java
similarity index 54%
copy from chrome/android/java/src/org/chromium/chrome/browser/media/router/ChromeMediaRouter.java
copy to chrome/android/java/src/org/chromium/chrome/browser/media/router/ChromeMediaRouterDialogController.java
index 9a87ef7e9667f5d773f87e1f2870198a40385814..1b3a473bbbe900fb7f229bd5a613eff34132bb31 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/media/router/ChromeMediaRouter.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/media/router/ChromeMediaRouterDialogController.java
@@ -10,23 +10,25 @@ import org.chromium.base.annotations.CalledByNative;
import org.chromium.base.annotations.JNINamespace;
/**
- * Implements the JNI interface called from the C++ Media Router implementation on Android.
+ * Implements the JNI interface called from the C++ Media Router dialog controller implementation
+ * on Android.
*/
@JNINamespace("media_router")
-public class ChromeMediaRouter {
+public class ChromeMediaRouterDialogController {
Bernhard Bauer 2015/08/06 15:23:01 What's the purpose of this class, other than, well
whywhat 2015/08/06 15:27:13 To show/hide media router dialog to select present
private static final String TAG = "cr.MediaRouter";
- private final long mNativeMediaRouter;
+ private final long mNativeDialogController;
/**
* Initializes the media router and its providers.
*/
@CalledByNative
- public static ChromeMediaRouter create(long nativeMediaRouter, Context context) {
- return new ChromeMediaRouter(nativeMediaRouter, context);
+ public static ChromeMediaRouterDialogController create(
+ long nativeDialogController, Context context) {
+ return new ChromeMediaRouterDialogController(nativeDialogController, context);
}
- private ChromeMediaRouter(long nativeMediaRouter, Context context) {
- mNativeMediaRouter = nativeMediaRouter;
+ private ChromeMediaRouterDialogController(long nativeDialogController, Context context) {
+ mNativeDialogController = nativeDialogController;
}
-}
+}

Powered by Google App Engine
This is Rietveld 408576698