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

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

Issue 2784353002: Android: Remove GetApplicationContext part 2 (Closed)
Patch Set: Fix tests Created 3 years, 8 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/ChromeMediaRouterDialogController.java b/chrome/android/java/src/org/chromium/chrome/browser/media/router/ChromeMediaRouterDialogController.java
index 4d21c7d96e49e573d48db8865f04a9d5ceebdca2..7217bb8b590967c25620f9bddaa37ee1f6bb2a47 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/media/router/ChromeMediaRouterDialogController.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/media/router/ChromeMediaRouterDialogController.java
@@ -4,7 +4,6 @@
package org.chromium.chrome.browser.media.router;
-import android.content.Context;
import android.support.v7.app.MediaRouteChooserDialogFragment;
import android.support.v7.app.MediaRouteControllerDialogFragment;
@@ -24,19 +23,16 @@ public class ChromeMediaRouterDialogController implements MediaRouteDialogDelega
"android.support.v7.mediarouter:MediaRouteControllerDialogFragment";
private final long mNativeDialogController;
- private final Context mApplicationContext;
private MediaRouteDialogManager mDialogManager;
/**
* Returns a new initialized {@link ChromeMediaRouterDialogController}.
* @param nativeDialogController the handle of the native object.
- * @param context the application context.
* @return a new dialog controller to use from the native side.
*/
@CalledByNative
- public static ChromeMediaRouterDialogController create(
- long nativeDialogController, Context context) {
- return new ChromeMediaRouterDialogController(nativeDialogController, context);
+ public static ChromeMediaRouterDialogController create(long nativeDialogController) {
+ return new ChromeMediaRouterDialogController(nativeDialogController);
}
/**
@@ -50,7 +46,7 @@ public class ChromeMediaRouterDialogController implements MediaRouteDialogDelega
MediaSource source = MediaSource.from(sourceUrn);
if (source == null) return;
- mDialogManager = new MediaRouteChooserDialogManager(source, mApplicationContext, this);
+ mDialogManager = new MediaRouteChooserDialogManager(source, this);
mDialogManager.openDialog();
}
@@ -66,8 +62,7 @@ public class ChromeMediaRouterDialogController implements MediaRouteDialogDelega
MediaSource source = MediaSource.from(sourceUrn);
if (source == null) return;
- mDialogManager = new MediaRouteControllerDialogManager(
- source, mediaRouteId, mApplicationContext, this);
+ mDialogManager = new MediaRouteControllerDialogManager(source, mediaRouteId, this);
mDialogManager.openDialog();
}
@@ -114,9 +109,8 @@ public class ChromeMediaRouterDialogController implements MediaRouteDialogDelega
nativeOnDialogCancelled(mNativeDialogController);
}
- private ChromeMediaRouterDialogController(long nativeDialogController, Context context) {
+ private ChromeMediaRouterDialogController(long nativeDialogController) {
mNativeDialogController = nativeDialogController;
- mApplicationContext = context;
}
native void nativeOnDialogCancelled(long nativeMediaRouterDialogControllerAndroid);

Powered by Google App Engine
This is Rietveld 408576698