Reload page when A11y is on and the JavaScript state changes.

Bug: 6910403
Change-Id: I0965d76e76282736d053947eb96be0af32fd6001

Author: alanv

core/java/android/webkit/AccessibilityInjector.java

@@ -279,6 +279,7 @@ public void onPageFinished(String url) {
         }
 
         if (!shouldInjectJavaScript(url)) {
+            mAccessibilityScriptInjected = false;
             toggleFallbackAccessibilityInjector(true);
             return;
         }
@@ -291,6 +292,23 @@ public void onPageFinished(String url) {
         mAccessibilityScriptInjected = true;
     }
 
+    /**
+     * Adjusts the accessibility injection state to reflect changes in the
+     * JavaScript enabled state.
+     *
+     * @param enabled Whether JavaScript is enabled.
+     */
+    public void updateJavaScriptEnabled(boolean enabled) {
+        if (enabled) {
+            addAccessibilityApisIfNecessary();
+        } else {
+            removeAccessibilityApisIfNecessary();
+        }
+
+        // We have to reload the page after adding or removing APIs.
+        mWebView.reload();
+    }
+
     /**
      * Toggles the non-JavaScript method for handling accessibility.
      *

core/java/android/webkit/WebSettingsClassic.java

@@ -1135,6 +1135,7 @@ public synchronized void setJavaScriptEnabled(boolean flag) {
         if (mJavaScriptEnabled != flag) {
             mJavaScriptEnabled = flag;
             postSync();
+            mWebView.updateJavaScriptEnabled(flag);
         }
     }
 

core/java/android/webkit/WebViewClassic.java

@@ -1630,6 +1630,12 @@ void updateMultiTouchSupport(Context context) {
         mZoomManager.updateMultiTouchSupport(context);
     }
 
+    void updateJavaScriptEnabled(boolean enabled) {
+        if (isAccessibilityEnabled()) {
+            getAccessibilityInjector().updateJavaScriptEnabled(enabled);
+        }
+    }
+
     private void init() {
         OnTrimMemoryListener.init(mContext);
         mWebView.setWillNotDraw(false);