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

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

Issue 1874423002: 🍰 Migrate app shared preferences to ContextUtils. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. Created 4 years, 7 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/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 8c5121d4e07a7b6271bc5f5159bdaa28c9458bc7..ab862b407b4376e73d91ffc9d54e8aad024afc06 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
@@ -8,10 +8,10 @@ import android.app.Activity;
import android.content.Context;
import android.content.Intent;
import android.os.Bundle;
-import android.preference.PreferenceManager;
import android.text.TextUtils;
import org.chromium.base.CommandLine;
+import org.chromium.base.ContextUtils;
import org.chromium.base.Log;
import org.chromium.base.VisibleForTesting;
import org.chromium.base.metrics.RecordUserAction;
@@ -129,7 +129,7 @@ public final class FirstRunSignInProcessor {
*/
@VisibleForTesting
public static boolean getFirstRunFlowSignInComplete(Context context) {
- return PreferenceManager.getDefaultSharedPreferences(context)
+ return ContextUtils.getAppSharedPreferences()
.getBoolean(FIRST_RUN_FLOW_SIGNIN_COMPLETE, false);
}
@@ -140,7 +140,7 @@ public final class FirstRunSignInProcessor {
*/
@VisibleForTesting
public static void setFirstRunFlowSignInComplete(Context context, boolean isComplete) {
- PreferenceManager.getDefaultSharedPreferences(context)
+ ContextUtils.getAppSharedPreferences()
.edit()
.putBoolean(FIRST_RUN_FLOW_SIGNIN_COMPLETE, isComplete)
.apply();
@@ -151,7 +151,7 @@ public final class FirstRunSignInProcessor {
* @param context A context
*/
private static String getFirstRunFlowSignInAccountName(Context context) {
- return PreferenceManager.getDefaultSharedPreferences(context)
+ return ContextUtils.getAppSharedPreferences()
.getString(FIRST_RUN_FLOW_SIGNIN_ACCOUNT_NAME, null);
}
@@ -161,7 +161,7 @@ public final class FirstRunSignInProcessor {
* @param accountName The account name, or null.
*/
private static void setFirstRunFlowSignInAccountName(Context context, String accountName) {
- PreferenceManager.getDefaultSharedPreferences(context)
+ ContextUtils.getAppSharedPreferences()
.edit()
.putString(FIRST_RUN_FLOW_SIGNIN_ACCOUNT_NAME, accountName)
.apply();
@@ -172,7 +172,7 @@ public final class FirstRunSignInProcessor {
* @param context A context
*/
private static boolean getFirstRunFlowSignInSetup(Context context) {
- return PreferenceManager.getDefaultSharedPreferences(context).getBoolean(
+ return ContextUtils.getAppSharedPreferences().getBoolean(
FIRST_RUN_FLOW_SIGNIN_SETUP, false);
}
@@ -182,7 +182,7 @@ public final class FirstRunSignInProcessor {
* @param isComplete Whether the user selected to see the settings once signed in.
*/
private static void setFirstRunFlowSignInSetup(Context context, boolean isComplete) {
- PreferenceManager.getDefaultSharedPreferences(context)
+ ContextUtils.getAppSharedPreferences()
.edit()
.putBoolean(FIRST_RUN_FLOW_SIGNIN_SETUP, isComplete)
.apply();

Powered by Google App Engine
This is Rietveld 408576698