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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/util/AccessibilityUtil.java

Issue 2826203003: Migrate java callers to always use AccessibilityUtil#isAccessibilityEnabled (Closed)
Patch Set: Created 3 years, 8 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/util/AccessibilityUtil.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/util/AccessibilityUtil.java b/chrome/android/java/src/org/chromium/chrome/browser/util/AccessibilityUtil.java
index 9d6368df6a5ba54836d98f2d5360d696d5dc3cd3..9d4a6ce167c56335f3b9d19a3dbfb85245c9ec53 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/util/AccessibilityUtil.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/util/AccessibilityUtil.java
@@ -12,7 +12,9 @@ import android.net.Uri;
import android.support.v7.app.AlertDialog;
import android.view.accessibility.AccessibilityManager;
+import org.chromium.base.ContextUtils;
import org.chromium.base.PackageUtils;
+import org.chromium.base.TraceEvent;
import org.chromium.chrome.R;
import java.util.List;
@@ -40,13 +42,17 @@ public class AccessibilityUtil {
/**
* Checks to see that this device has accessibility and touch exploration enabled.
- * @param context A {@link Context} instance.
* @return Whether or not accessibility and touch exploration are enabled.
*/
- public static boolean isAccessibilityEnabled(Context context) {
- AccessibilityManager manager = (AccessibilityManager)
- context.getSystemService(Context.ACCESSIBILITY_SERVICE);
- return manager != null && manager.isEnabled() && manager.isTouchExplorationEnabled();
+ public static boolean isAccessibilityEnabled() {
+ TraceEvent.begin("AccessibilityManager::isAccessibilityEnabled");
+ AccessibilityManager manager =
+ (AccessibilityManager) ContextUtils.getApplicationContext().getSystemService(
+ Context.ACCESSIBILITY_SERVICE);
+ boolean retVal =
+ manager != null && manager.isEnabled() && manager.isTouchExplorationEnabled();
+ TraceEvent.end("AccessibilityManager::isAccessibilityEnabled");
+ return retVal;
}
/**

Powered by Google App Engine
This is Rietveld 408576698