diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/AccessibilityDelegateUtil.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/AccessibilityDelegateUtil.java index 289042cd479..63e67656989 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/AccessibilityDelegateUtil.java +++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/AccessibilityDelegateUtil.java @@ -1,20 +1,16 @@ -// Copyright (c) 2004-present, Facebook, Inc. +// Copyright (c) Facebook, Inc. and its affiliates. // This source code is licensed under the MIT license found in the // LICENSE file in the root directory of this source tree. package com.facebook.react.uimanager; -import android.annotation.TargetApi; import android.content.Context; -import android.os.Build; import android.support.v4.view.AccessibilityDelegateCompat; import android.support.v4.view.ViewCompat; import android.support.v4.view.accessibility.AccessibilityNodeInfoCompat; import android.view.View; -import android.view.accessibility.AccessibilityNodeInfo; import com.facebook.react.R; -import com.facebook.react.bridge.ReadableArray; import java.util.Locale; import javax.annotation.Nullable; @@ -76,11 +72,11 @@ public class AccessibilityDelegateUtil { public static AccessibilityRole fromValue(@Nullable String value) { for (AccessibilityRole role : AccessibilityRole.values()) { - if (role.getValue() != null && role.getValue().equals(value)) { + if (role.name().equalsIgnoreCase(value)) { return role; } } - return AccessibilityRole.NONE; + throw new IllegalArgumentException("Invalid accessibility role value: " + value); } } @@ -149,14 +145,4 @@ public class AccessibilityDelegateUtil { nodeInfo.setClickable(true); } } - - /** - * Method for setting accessibilityRole on view properties. - */ - public static AccessibilityRole getAccessibilityRole(String role) { - if (role == null) { - return AccessibilityRole.NONE; - } - return AccessibilityRole.valueOf(role.toUpperCase()); - } -} +} \ No newline at end of file diff --git a/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java b/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java index fd5b2de97f6..e05d85dbfa0 100644 --- a/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java +++ b/ReactAndroid/src/main/java/com/facebook/react/uimanager/BaseViewManager.java @@ -11,6 +11,7 @@ import android.view.View; import android.view.ViewParent; import com.facebook.react.R; import com.facebook.react.bridge.ReadableArray; +import com.facebook.react.uimanager.AccessibilityDelegateUtil.AccessibilityRole; import com.facebook.react.uimanager.annotations.ReactProp; import com.facebook.react.uimanager.util.ReactFindViewUtil; import java.util.Locale; @@ -131,14 +132,8 @@ public abstract class BaseViewManager