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

Unified Diff: components/policy/android/java/src/org/chromium/policy/AbstractAppRestrictionsProvider.java

Issue 1965093002: 🎾 Migrate more 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: components/policy/android/java/src/org/chromium/policy/AbstractAppRestrictionsProvider.java
diff --git a/components/policy/android/java/src/org/chromium/policy/AbstractAppRestrictionsProvider.java b/components/policy/android/java/src/org/chromium/policy/AbstractAppRestrictionsProvider.java
index 64f886b169d84764d0a02b7cdcf3b3c01a8bf122..311bba610b8f45db36acb341bb59752152cda5d4 100644
--- a/components/policy/android/java/src/org/chromium/policy/AbstractAppRestrictionsProvider.java
+++ b/components/policy/android/java/src/org/chromium/policy/AbstractAppRestrictionsProvider.java
@@ -8,14 +8,13 @@ import android.content.BroadcastReceiver;
import android.content.Context;
import android.content.Intent;
import android.content.IntentFilter;
-import android.content.SharedPreferences;
import android.os.AsyncTask;
import android.os.Bundle;
import android.os.Handler;
import android.os.Parcel;
-import android.preference.PreferenceManager;
import android.util.Base64;
+import org.chromium.base.ContextUtils;
import org.chromium.base.Log;
import org.chromium.base.ThreadUtils;
import org.chromium.base.VisibleForTesting;
@@ -39,7 +38,6 @@ public abstract class AbstractAppRestrictionsProvider extends PolicyProvider {
private static Bundle sTestRestrictions = null;
private final Context mContext;
- private final SharedPreferences mSharedPreferences;
private final BroadcastReceiver mAppRestrictionsChangedReceiver = new BroadcastReceiver() {
@Override
public void onReceive(Context context, Intent intent) {
@@ -54,7 +52,6 @@ public abstract class AbstractAppRestrictionsProvider extends PolicyProvider {
*/
public AbstractAppRestrictionsProvider(Context context) {
mContext = context;
- mSharedPreferences = PreferenceManager.getDefaultSharedPreferences(mContext);
}
/**
@@ -138,13 +135,11 @@ public abstract class AbstractAppRestrictionsProvider extends PolicyProvider {
p.writeBundle(policies);
byte bytes[] = p.marshall();
String s = Base64.encodeToString(bytes, 0);
- SharedPreferences.Editor ed = mSharedPreferences.edit();
- ed.putString(PREFERENCE_KEY, s);
- ed.apply();
+ ContextUtils.getAppSharedPreferences().edit().putString(PREFERENCE_KEY, s).apply();
}
private Bundle getCachedPolicies() {
- String s = mSharedPreferences.getString(PREFERENCE_KEY, null);
+ String s = ContextUtils.getAppSharedPreferences().getString(PREFERENCE_KEY, null);
if (s == null) {
return null;
}

Powered by Google App Engine
This is Rietveld 408576698