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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/accessibility/FontSizePrefsTest.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/javatests/src/org/chromium/chrome/browser/accessibility/FontSizePrefsTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/accessibility/FontSizePrefsTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/accessibility/FontSizePrefsTest.java
new file mode 100644
index 0000000000000000000000000000000000000000..aa0211136ca8ad6f5945e2d5c6daf9cb4e1c380b
--- /dev/null
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/accessibility/FontSizePrefsTest.java
@@ -0,0 +1,170 @@
+// 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.test.suitebuilder.annotation.SmallTest;
+
+import org.chromium.base.test.util.Feature;
+import org.chromium.base.ThreadUtils;
+import org.chromium.chrome.browser.profiles.Profile;
+import org.chromium.chrome.shell.ChromeShellTestBase;
+import org.chromium.content.browser.test.util.UiUtils;
+
+/**
+ * Test class for {@link FontSizePrefs}.
+ */
+public class FontSizePrefsTest extends ChromeShellTestBase {
+
+ private FontSizePrefs mFontSizePrefs;
+
+ @Override
+ public void setUp() throws Exception {
+ super.setUp();
+ startChromeBrowserProcessSync(getInstrumentation().getTargetContext());
+ mFontSizePrefs = FontSizePrefs.getForProfile(Profile.getLastUsedProfile(),
+ launchChromeShellWithBlankPage().getApplicationContext());
+ }
+
+ @SmallTest
+ @Feature({"Accessibility"})
+ public void testGetAndSetFontAndForceEnableZoom() throws InterruptedException {
+ // Check default font.
+ assertEquals(1f, mFontSizePrefs.getFontScaleFactor());
+ // Check that setting the value of font scale factor works.
+ float newTextScale = 1.2f;
+ setFontScale(newTextScale);
+ assertEquals(newTextScale, mFontSizePrefs.getFontScaleFactor());
+
+ // Check the default value of force enable zoom.
+ assertFalse(mFontSizePrefs.getForceEnableZoom());
+ // Check that setting the value of force enable zoom works.
+ setForceEnableZoom(true);
+ assertTrue(mFontSizePrefs.getForceEnableZoom());
+ }
+
+ @SmallTest
+ @Feature({"Accessibility"})
+ public void testGetAndSetUserSetForceEnableZoom() throws InterruptedException {
+ // Check the default value of user set force enable zoom.
+ assertFalse(mFontSizePrefs.getUserSetForceEnableZoom());
+ // Check that setting the value of user set force enable zoom works.
+ setUserSetForceEnableZoom(true);
+ assertTrue(mFontSizePrefs.getUserSetForceEnableZoom());
+ }
+
+ @SmallTest
+ @Feature({"Accessibility"})
+ public void testObserversForceEnableZoom() throws InterruptedException {
+ TestingObserver test1 = new TestingObserver();
+ TestingObserver test2 = new TestingObserver();
+ mFontSizePrefs.addObserver(test1);
+ mFontSizePrefs.addObserver(test2);
+
+ // Checks that force enable zoom for both observers is correctly changed.
+ setForceEnableZoom(true);
+ UiUtils.settleDownUI(getInstrumentation());
+ assertTrue(test1.getForceEnableZoom());
+ assertTrue(test2.getForceEnableZoom());
+
+ // Checks that removing observer and setting force enable zoom works.
+ mFontSizePrefs.removeObserver(test1);
+ setForceEnableZoom(false);
+ UiUtils.settleDownUI(getInstrumentation());
+ assertTrue(test1.getForceEnableZoom());
+ assertFalse(test2.getForceEnableZoom());
+ mFontSizePrefs.removeObserver(test2);
+ }
+
+ @SmallTest
+ @Feature({"Accessibility"})
+ public void testObserversFontScale() throws InterruptedException {
+ TestingObserver test1 = new TestingObserver();
+ TestingObserver test2 = new TestingObserver();
+ mFontSizePrefs.addObserver(test1);
+ mFontSizePrefs.addObserver(test2);
+
+ // Checks that font scale for both observers is correctly changed.
+ float newTextScale = 1.2f;
+ setFontScale(newTextScale);
+ UiUtils.settleDownUI(getInstrumentation());
+ assertEquals(newTextScale, test1.getFontScaleFactor());
+ assertEquals(newTextScale, test2.getFontScaleFactor());
+
+ // Checks that removing observer and setting font works.
+ float newerTextScale = 1.4f;
+ mFontSizePrefs.removeObserver(test1);
+ setFontScale(newerTextScale);
+ UiUtils.settleDownUI(getInstrumentation());
+ assertEquals(newTextScale, test1.getFontScaleFactor());
+ assertEquals(newerTextScale, test2.getFontScaleFactor());
+ mFontSizePrefs.removeObserver(test2);
+ }
+
+ @SmallTest
+ @Feature({"Accessibility"})
+ public void testMultipleAddMultipleDeleteObservers() throws InterruptedException {
+ TestingObserver test = new TestingObserver();
+
+ // Should successfully add the observer the first time.
+ assertTrue(mFontSizePrefs.addObserver(test));
+ // Observer cannot be added again, should return false.
+ assertFalse(mFontSizePrefs.addObserver(test));
+
+ // Delete the observer the first time.
+ assertTrue(mFontSizePrefs.removeObserver(test));
+ // Observer cannot be deleted again, should return false.
+ assertFalse(mFontSizePrefs.removeObserver(test));
+ }
+
+ private static class TestingObserver implements FontSizePrefs.Observer {
+ private float mFontSize;
+ private boolean mForceEnableZoom;
+
+ public TestingObserver() {
+ mFontSize = 1;
+ mForceEnableZoom = false;
+ }
+
+ public float getFontScaleFactor() {
+ return mFontSize;
+ }
+
+ public void onChangeFontSize(float font) {
+ mFontSize = font;
+ }
+
+ public boolean getForceEnableZoom() {
+ return mForceEnableZoom;
+ }
+
+ public void onChangeForceEnableZoom(boolean enabled) {
+ mForceEnableZoom = enabled;
+ }
+ }
+
+ private void setFontScale(final float fontsize) {
+ ThreadUtils.runOnUiThreadBlocking(new Runnable() {
+ public void run() {
+ mFontSizePrefs.setFontScaleFactor(fontsize);
+ }
+ });
+ }
+
+ private void setForceEnableZoom(final boolean enabled) {
+ ThreadUtils.runOnUiThreadBlocking(new Runnable() {
+ public void run() {
+ mFontSizePrefs.setForceEnableZoom(enabled);
+ }
+ });
+ }
+
+ private void setUserSetForceEnableZoom(final boolean enabled) {
+ ThreadUtils.runOnUiThreadBlocking(new Runnable() {
+ public void run() {
+ mFontSizePrefs.setUserSetForceEnableZoom(enabled);
+ }
+ });
+ }
+}

Powered by Google App Engine
This is Rietveld 408576698