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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/media/router/MediaRouteChooserDialogManager.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
Index: chrome/android/java/src/org/chromium/chrome/browser/media/router/MediaRouteChooserDialogManager.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/media/router/MediaRouteChooserDialogManager.java b/chrome/android/java/src/org/chromium/chrome/browser/media/router/MediaRouteChooserDialogManager.java
index 0b5eb82d314c4c37182cf267f241678b3afd7aef..57bcb15e4434b2d50beeca1f1a27d0ed87c8b648 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/media/router/MediaRouteChooserDialogManager.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/media/router/MediaRouteChooserDialogManager.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.MediaRouteChooserDialogFragment;
import android.support.v7.media.MediaRouteSelector;
import org.chromium.chrome.browser.media.router.cast.MediaSink;
@@ -29,15 +31,35 @@ public class MediaRouteChooserDialogManager extends BaseMediaRouteDialogManager
/**
* Fragment implementation for MediaRouteChooserDialogManager.
*/
- public static class Fragment extends BaseMediaRouteDialogManager.Fragment {
+ public static class Fragment extends MediaRouteChooserDialogFragment {
+ private final Handler mHandler = new Handler();
+ private final SystemVisibilitySaver mVisibilitySaver = new SystemVisibilitySaver();
+ private BaseMediaRouteDialogManager mManager = null;
private boolean mCancelled;
public Fragment() {
- super();
+ mHandler.post(new Runnable() {
+ @Override
+ public void run() {
+ Fragment.this.dismiss();
+ }
+ });
}
public Fragment(BaseMediaRouteDialogManager manager) {
- super(manager);
+ mManager = manager;
+ }
+
+ @Override
+ public void onStart() {
+ mVisibilitySaver.saveSystemVisibility(getActivity());
+ super.onStart();
+ }
+
+ @Override
+ public void onStop() {
+ super.onStop();
+ mVisibilitySaver.restoreSystemVisibility(getActivity());
}
@Override
@@ -52,8 +74,11 @@ public class MediaRouteChooserDialogManager extends BaseMediaRouteDialogManager
@Override
public void onDismiss(DialogInterface dialog) {
super.onDismiss(dialog);
+ if (mManager == null) return;
+
+ mManager.mDialogFragment = null;
- if (mCancelled || mManager == null) return;
+ if (mCancelled) return;
MediaSink newSink =
MediaSink.fromRoute(mManager.androidMediaRouter().getSelectedRoute());

Powered by Google App Engine
This is Rietveld 408576698