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

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

Issue 2546463002: [MediaRouter, Android] Rebase the controller dialog fragment. (Closed)
Patch Set: Created 4 years 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
« no previous file with comments | « chrome/android/java/src/org/chromium/chrome/browser/media/router/MediaRouteChooserDialogManager.java ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/media/router/MediaRouteControllerDialogManager.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/media/router/MediaRouteControllerDialogManager.java b/chrome/android/java/src/org/chromium/chrome/browser/media/router/MediaRouteControllerDialogManager.java
index 5a9b510bd1f709cc9752c902379c8318b332dda5..65afa85d337d3c90fd1fdf47c2e5d713f833a588 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/media/router/MediaRouteControllerDialogManager.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/media/router/MediaRouteControllerDialogManager.java
@@ -6,8 +6,10 @@ package org.chromium.chrome.browser.media.router;
import android.content.Context;
import android.content.DialogInterface;
+import android.os.Handler;
import android.support.v4.app.DialogFragment;
import android.support.v4.app.FragmentManager;
+import android.support.v7.app.MediaRouteControllerDialogFragment;
import android.support.v7.media.MediaRouteSelector;
import android.support.v7.media.MediaRouter;
@@ -40,26 +42,46 @@ public class MediaRouteControllerDialogManager extends BaseMediaRouteDialogManag
/**
* Fragment implementation for MediaRouteControllerDialogManager.
*/
- public static class Fragment extends BaseMediaRouteDialogManager.Fragment {
- MediaRouter.Callback mCallback = null;
+ public static class Fragment extends MediaRouteControllerDialogFragment {
+ private final Handler mHandler = new Handler();
+ private final SystemVisibilitySaver mVisibilitySaver = new SystemVisibilitySaver();
+ private BaseMediaRouteDialogManager mManager = null;
+ private MediaRouter.Callback mCallback = null;
public Fragment() {
- super();
+ mHandler.post(new Runnable() {
+ @Override
+ public void run() {
+ Fragment.this.dismiss();
+ }
+ });
}
public Fragment(BaseMediaRouteDialogManager manager, MediaRouter.Callback callback) {
- super(manager);
+ mManager = manager;
mCallback = callback;
}
@Override
- public void onDismiss(DialogInterface dialog) {
- if (mManager != null) {
- mManager.delegate().onDialogCancelled();
- mManager.androidMediaRouter().removeCallback(mCallback);
- }
+ public void onStart() {
+ mVisibilitySaver.saveSystemVisibility(getActivity());
+ super.onStart();
+ }
+
+ @Override
+ public void onStop() {
+ super.onStop();
+ mVisibilitySaver.restoreSystemVisibility(getActivity());
+ }
+ @Override
+ public void onDismiss(DialogInterface dialog) {
super.onDismiss(dialog);
+ if (mManager == null) return;
+
+ mManager.delegate().onDialogCancelled();
+ mManager.androidMediaRouter().removeCallback(mCallback);
+ mManager.mDialogFragment = null;
}
};
« no previous file with comments | « chrome/android/java/src/org/chromium/chrome/browser/media/router/MediaRouteChooserDialogManager.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698