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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/signin/ConfirmSyncDataStateMachineDelegate.java

Issue 2772203004: Add progress and timeout dialogs for getting account management policy (Closed)
Patch Set: Updated strings and addressed comments Created 3 years, 9 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/signin/ConfirmSyncDataStateMachineDelegate.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/signin/ConfirmSyncDataStateMachineDelegate.java b/chrome/android/java/src/org/chromium/chrome/browser/signin/ConfirmSyncDataStateMachineDelegate.java
new file mode 100644
index 0000000000000000000000000000000000000000..b33ebdee1fbb24bb8334fab4399e86960a37da5d
--- /dev/null
+++ b/chrome/android/java/src/org/chromium/chrome/browser/signin/ConfirmSyncDataStateMachineDelegate.java
@@ -0,0 +1,127 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+package org.chromium.chrome.browser.signin;
+
+import android.app.Dialog;
+import android.content.Context;
+import android.content.DialogInterface;
+import android.support.v7.app.AlertDialog;
+
+import org.chromium.chrome.R;
+
+/**
+ * Class to decouple ConfirmSyncDataStateMachine from UI code and dialog management.
+ */
+public class ConfirmSyncDataStateMachineDelegate {
+ /**
+ * Listener to receive events from progress dialog. If the dialog is not dismissed by showing
+ * other dialog or calling {@link ConfirmSyncDataStateMachineDelegate#dismissAllDialogs},
+ * then {@link #onCancel} will be called once.
+ */
+ public interface ProgressDialogListener {
+ /**
+ * This method is called when user cancels the dialog in any way.
+ */
+ void onCancel();
+ }
+
+ /**
+ * Listener to receive events from timeout dialog. If the dialog is not dismissed by showing
+ * other dialog or calling {@link ConfirmSyncDataStateMachineDelegate#dismissAllDialogs},
+ * then either {@link #onCancel} or {@link #onRetry} will be called once.
+ */
+ public interface TimeoutDialogListener {
+ /**
+ * This method is called when user cancels the dialog in any way.
+ */
+ void onCancel();
+
+ /**
+ * This method is called when user clicks retry button.
+ */
+ void onRetry();
+ }
+
+ private final Context mContext;
+
+ private Dialog mProgressDialog;
+ private AlertDialog mTimeoutAlertDialog;
+
+ public ConfirmSyncDataStateMachineDelegate(Context context) {
+ mContext = context;
+ }
+
+ /**
+ * Shows progress dialog. Will dismiss other dialogs shown, if any.
+ *
+ * @param listener The {@link ProgressDialogListener} that will be notified about user actions.
+ */
+ public void showFetchManagementPolicyProgressDialog(final ProgressDialogListener listener) {
+ dismissAllDialogs();
+ mProgressDialog = new AlertDialog.Builder(mContext, R.style.AlertDialogTheme)
+ .setView(R.layout.signin_progress_bar_dialog)
+ .setNegativeButton(R.string.cancel,
+ new DialogInterface.OnClickListener() {
+ @Override
+ public void onClick(DialogInterface dialog, int i) {
+ dialog.cancel();
+ }
+ })
+ .setOnCancelListener(new DialogInterface.OnCancelListener() {
+ @Override
+ public void onCancel(DialogInterface dialog) {
+ listener.onCancel();
+ }
+ })
+ .create();
+ mProgressDialog.show();
+ }
+
+ /**
+ * Shows timeout dialog. Will dismiss other dialogs shown, if any.
+ *
+ * @param listener The {@link TimeoutDialogListener} that will be notified about user actions.
+ */
+ public void showFetchManagementPolicyTimeoutDialog(final TimeoutDialogListener listener) {
+ dismissAllDialogs();
+ mTimeoutAlertDialog =
+ new AlertDialog.Builder(mContext, R.style.AlertDialogTheme)
+ .setTitle(R.string.sign_in_timeout_title)
+ .setMessage(R.string.sign_in_timeout_message)
+ .setNegativeButton(R.string.cancel,
+ new DialogInterface.OnClickListener() {
+ @Override
+ public void onClick(DialogInterface dialog, int which) {
+ dialog.cancel();
+ }
+ })
+ .setPositiveButton(R.string.retry,
+ new DialogInterface.OnClickListener() {
+ @Override
+ public void onClick(DialogInterface dialog, int which) {
+ listener.onRetry();
+ }
+ })
+ .setOnCancelListener(new DialogInterface.OnCancelListener() {
+ @Override
+ public void onCancel(DialogInterface dialog) {
+ listener.onCancel();
+ }
+ })
+ .create();
+ mTimeoutAlertDialog.show();
+ }
+
+ /**
+ * Dismisses all dialogs.
+ */
+ public void dismissAllDialogs() {
+ if (mProgressDialog != null) mProgressDialog.dismiss();
+ mProgressDialog = null;
+
+ if (mTimeoutAlertDialog != null) mTimeoutAlertDialog.dismiss();
+ mTimeoutAlertDialog = null;
+ }
+}

Powered by Google App Engine
This is Rietveld 408576698