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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/accessibility/FontSizePrefs.java

Issue 415343002: Upstream accessibility font size preferences. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: brackets of return statement Created 6 years, 5 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/accessibility/FontSizePrefs.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/accessibility/FontSizePrefs.java b/chrome/android/java/src/org/chromium/chrome/browser/accessibility/FontSizePrefs.java
new file mode 100644
index 0000000000000000000000000000000000000000..a9721f5e7b89a9e741ad95e9cda5397dd8d78bd8
--- /dev/null
+++ b/chrome/android/java/src/org/chromium/chrome/browser/accessibility/FontSizePrefs.java
@@ -0,0 +1,184 @@
+// Copyright 2014 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.accessibility;
+
+import android.content.Context;
+import android.content.SharedPreferences;
+import android.preference.PreferenceManager;
+
+import org.chromium.base.CalledByNative;
+import org.chromium.base.NativeCall;
+import org.chromium.base.ThreadUtils;
+import org.chromium.chrome.browser.profiles.Profile;
+
+import java.util.HashMap;
+import java.util.Map;
+
+/**
+ * Interface for Font Scale Factor, Force Enable Zoom, and User Set Force Enable Zoom preferences.
+ */
+public class FontSizePrefs {
+ public static final String PREF_FORCE_ENABLE_ZOOM = "force_enable_zoom";
+ public static final String PREF_TEXT_SCALE = "text_scale";
+ public static final String PREF_USER_SET_FORCE_ENABLE_ZOOM = "user_set_force_enable_zoom";
+
+ private final long mFontSizePrefsAndroidPtr;
+ private final Profile mProfile;
+ private final SharedPreferences mSharedPreferences;
+ private final Map<Observer, FontSizePrefsObserverWrapper> mObserverMap;
+ private static HashMap<Profile, FontSizePrefs> sFontSizeMap =
robliao 2014/08/01 17:36:33 Make this final.
sunangel 2014/08/04 16:37:56 Done.
+ new HashMap<Profile, FontSizePrefs>();
+
+ /*
+ * Observer interface for observing changes in FontScaleFactor and ForceEnableZoom. Note that
+ * this observer does not observe changes in UserSetForceEnableZoom.
+ */
+ public interface Observer {
+ void onChangeFontSize(float newFontSize);
+ void onChangeForceEnableZoom(boolean enabled);
+ }
+
+ /*
+ * Wrapper for FontSizePrefsObserverAndroid.
+ */
+ public class FontSizePrefsObserverWrapper {
+ private final Observer mFontSizePrefsObserver;
+ private final long mNativeFontSizePrefsObserverWrapperPtr;
+
+ public FontSizePrefsObserverWrapper(Observer observer) {
+ mNativeFontSizePrefsObserverWrapperPtr = nativeInitObserverAndroid();
+ mFontSizePrefsObserver = observer;
+ }
+
+ public long getNativePtr() {
+ return mNativeFontSizePrefsObserverWrapperPtr;
+ }
+
+ public void destroy() {
+ nativeDestroyObserverAndroid(mNativeFontSizePrefsObserverWrapperPtr);
+ }
+
+ @CalledByNative("FontSizePrefsObserverWrapper")
+ public void onChangeFontSize(float newFontSize) {
+ mFontSizePrefsObserver.onChangeFontSize(newFontSize);
+ }
+
+ @CalledByNative("FontSizePrefsObserverWrapper")
+ public void onChangeForceEnableZoom(boolean enabled) {
+ mFontSizePrefsObserver.onChangeForceEnableZoom(enabled);
+ }
+
+ @NativeCall("FontSizePrefsObserverWrapper")
+ private native long nativeInitObserverAndroid();
+
+ @NativeCall("FontSizePrefsObserverWrapper")
+ private native void nativeDestroyObserverAndroid(long nativeFontSizePrefsObserverAndroid);
+ }
+
+ public FontSizePrefs(Profile profile, Context context) {
+ mProfile = profile;
+ mSharedPreferences = PreferenceManager.getDefaultSharedPreferences(context);
+ mFontSizePrefsAndroidPtr = nativeInit(profile);
+ mObserverMap = new HashMap<Observer, FontSizePrefsObserverWrapper>();
+ }
+
+ /*
+ * Returns the FontSizePrefs corresponding to the inputted Profile. If no FontSizePrefs existed,
+ * this method will create one.
+ */
+ public static FontSizePrefs getForProfile(Profile profile, Context context) {
+ ThreadUtils.assertOnUiThread();
+ FontSizePrefs fontSizePrefs = sFontSizeMap.get(profile);
+ if (fontSizePrefs == null) {
+ fontSizePrefs = new FontSizePrefs(profile, context);
+ sFontSizeMap.put(profile, fontSizePrefs);
+ }
+ return fontSizePrefs;
+ }
+
+ /*
+ * Adds the observer to listen for Font Scale and Force Enable Zoom preferences.
+ * @return true if the observerMap was changed as a result of the call.
+ */
+ public boolean addObserver(Observer obs) {
+ if (mObserverMap.containsKey(obs)) return false;
+ FontSizePrefsObserverWrapper wrappedObserver = new
robliao 2014/08/01 17:36:33 new should go on next line.
robliao 2014/08/01 17:36:33 New should go on next line.
sunangel 2014/08/04 16:37:56 Done.
+ FontSizePrefsObserverWrapper(obs);
+ nativeAddObserver(mFontSizePrefsAndroidPtr, wrappedObserver.getNativePtr());
+ mObserverMap.put(obs, wrappedObserver);
+ return true;
+ }
+
+ /*
+ * Removes the observer and unregisters it from Font Scale and Force Enable Zoom changes.
+ * @return true if an observer was removed as a result of the call.
+ */
+ public boolean removeObserver(Observer obs) {
+ FontSizePrefsObserverWrapper wrappedObserver = mObserverMap.remove(obs);
+ if (wrappedObserver != null) {
+ nativeRemoveObserver(mFontSizePrefsAndroidPtr, wrappedObserver.getNativePtr());
+ wrappedObserver.destroy();
+ return true;
+ }
+ return false;
+ }
+
+ /*
+ * Sets UserSetForceEnableZoom. This is the only one of three preferences stored through
+ * SharedPreferences.
+ */
+ public void setUserSetForceEnableZoom(boolean enabled) {
+ SharedPreferences.Editor sharedPreferencesEditor = mSharedPreferences.edit();
+ sharedPreferencesEditor.putBoolean(PREF_USER_SET_FORCE_ENABLE_ZOOM, enabled);
+ sharedPreferencesEditor.apply();
+ }
+
+ /*
+ * Returns true if user has manually set ForceEnableZoom and false otherwise.
+ */
+ public boolean getUserSetForceEnableZoom() {
+ return mSharedPreferences.getBoolean(PREF_USER_SET_FORCE_ENABLE_ZOOM,
+ false);
+ }
+
+ public void setFontScaleFactor(float fontScaleFactor) {
+ nativeSetFontScaleFactor(mFontSizePrefsAndroidPtr, fontScaleFactor);
+ }
+
+ public float getFontScaleFactor() {
+ return nativeGetFontScaleFactor(mFontSizePrefsAndroidPtr);
+ }
+
+ public void setForceEnableZoom(boolean enabled) {
+ nativeSetForceEnableZoom(mFontSizePrefsAndroidPtr, enabled);
+ }
+
+ public boolean getForceEnableZoom() {
+ return nativeGetForceEnableZoom(mFontSizePrefsAndroidPtr);
+ }
+
+ public void destroy() {
+ sFontSizeMap.remove(mProfile);
+ nativeDestroy(mFontSizePrefsAndroidPtr);
+ }
+
+ private native void nativeAddObserver(long nativeFontSizePrefsAndroid,
+ long nativeObserverPtr);
+
+ private native void nativeRemoveObserver(long nativeFontSizePrefsAndroid,
+ long nativeObserverPtr);
+
+ private native long nativeInit(Profile profile);
+
+ private native void nativeDestroy(long nativeFontSizePrefsAndroid);
+
+ private native void nativeSetFontScaleFactor(long nativeFontSizePrefsAndroid, float font);
+
+ private native float nativeGetFontScaleFactor(long nativeFontSizePrefsAndroid);
+
+ private native boolean nativeGetForceEnableZoom(long nativeFontSizePrefsAndroid);
+
+ private native void nativeSetForceEnableZoom(long nativeFontSizePrefsAndroid, boolean enabled);
+}

Powered by Google App Engine
This is Rietveld 408576698