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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/firstrun/FirstRunSignInProcessor.java

Issue 2566963002: Revert of Disable UI options that allow new tabs before FRE. (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/firstrun/FirstRunSignInProcessor.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/firstrun/FirstRunSignInProcessor.java b/chrome/android/java/src/org/chromium/chrome/browser/firstrun/FirstRunSignInProcessor.java
index b5237c5811b77d941fe188775bf6a1bf5769732e..47d230acef5242414773c7db5e6937ac16dbb4c9 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/firstrun/FirstRunSignInProcessor.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/firstrun/FirstRunSignInProcessor.java
@@ -58,7 +58,7 @@
SigninManager signinManager = SigninManager.get(activity.getApplicationContext());
signinManager.onFirstRunCheckDone();
- boolean firstRunFlowComplete = FirstRunStatus.getFirstRunFlowComplete();
+ boolean firstRunFlowComplete = FirstRunStatus.getFirstRunFlowComplete(activity);
// We skip signin and the FRE if
// - FRE is disabled, or
// - FRE hasn't been completed, but the user has already seen the ToS in the Setup Wizard.
@@ -129,7 +129,7 @@
Log.e(TAG, "Attempt to pass-through without completed FRE");
// Things went wrong -- we want the user to go through the full FRE.
- FirstRunStatus.setFirstRunFlowComplete(false);
+ FirstRunStatus.setFirstRunFlowComplete(activity, false);
setFirstRunFlowSignInComplete(activity, false);
setFirstRunFlowSignInAccountName(activity, null);
setFirstRunFlowSignInSetup(activity, false);
@@ -207,7 +207,7 @@
* @param data Resulting FRE properties bundle
*/
public static void finalizeFirstRunFlowState(Context context, Bundle data) {
- FirstRunStatus.setFirstRunFlowComplete(true);
+ FirstRunStatus.setFirstRunFlowComplete(context, true);
setFirstRunFlowSignInAccountName(context,
data.getString(FirstRunActivity.RESULT_SIGNIN_ACCOUNT_NAME));
setFirstRunFlowSignInSetup(
@@ -221,7 +221,7 @@
public static void updateSigninManagerFirstRunCheckDone(Context context) {
SigninManager manager = SigninManager.get(context);
if (manager.isSignInAllowed()) return;
- if (!FirstRunStatus.getFirstRunFlowComplete()) return;
+ if (!FirstRunStatus.getFirstRunFlowComplete(context)) return;
if (!getFirstRunFlowSignInComplete(context)) return;
manager.onFirstRunCheckDone();
}

Powered by Google App Engine
This is Rietveld 408576698