ensure a11y state is communicated back to flutter (#5631)

Jonah Williams · web-flow · commit ce06bbac58d7 · 2018-06-28T13:30:51.000-07:00
diff --git a/shell/platform/android/io/flutter/view/FlutterView.java b/shell/platform/android/io/flutter/view/FlutterView.java
@@ -808,6 +808,7 @@ public void onAccessibilityStateChanged(boolean enabled) {
             if (mAccessibilityNodeProvider != null) {
                 mAccessibilityNodeProvider.setAccessibilityEnabled(false);
             }
+            nativeSetSemanticsEnabled(mNativeView.get(), false);
         }
         resetWillNotDraw();
     }
@@ -846,8 +847,8 @@ void ensureAccessibilityEnabled() {
         mAccessibilityEnabled = true;
         if (mAccessibilityNodeProvider == null) {
             mAccessibilityNodeProvider = new AccessibilityBridge(this);
-            nativeSetSemanticsEnabled(mNativeView.get(), true);
         }
+        nativeSetSemanticsEnabled(mNativeView.get(), true);
         mAccessibilityNodeProvider.setAccessibilityEnabled(true);
     }
 

Original file line number	Diff line number	Diff line change
`@@ -808,6 +808,7 @@ public void onAccessibilityStateChanged(boolean enabled) {`
`808`	`808`	`if (mAccessibilityNodeProvider != null) {`
`809`	`809`	`mAccessibilityNodeProvider.setAccessibilityEnabled(false);`
`810`	`810`	`}`
	`811`	`+ nativeSetSemanticsEnabled(mNativeView.get(), false);`
`811`	`812`	`}`
`812`	`813`	`resetWillNotDraw();`
`813`	`814`	`}`
`@@ -846,8 +847,8 @@ void ensureAccessibilityEnabled() {`
`846`	`847`	`mAccessibilityEnabled = true;`
`847`	`848`	`if (mAccessibilityNodeProvider == null) {`
`848`	`849`	`mAccessibilityNodeProvider = new AccessibilityBridge(this);`
`849`		`- nativeSetSemanticsEnabled(mNativeView.get(), true);`
`850`	`850`	`}`
	`851`	`+ nativeSetSemanticsEnabled(mNativeView.get(), true);`
`851`	`852`	`mAccessibilityNodeProvider.setAccessibilityEnabled(true);`
`852`	`853`	`}`
`853`	`854`