Remove AccessibilityInjector support.

Remove AccessibilityInjector support.

AccessibilityInjector is no longer used as it was needed for ICS support
and Chrome only support API 16 and above now.

BUG=479334
TBR=damienv@chromium.org

Committed: https://crrev.com/eeffed48a0847161775df9cec591446b80c83e54
Cr-Commit-Position: refs/heads/master@{#326419}

[aurimas (slooooooooow), 2015-04-21 22:20:40]

aurimas@chromium.org changed reviewers:
+ dmazzoni@chromium.org, dtrainor@chromium.org

[aurimas (slooooooooow), 2015-04-21 22:20:41]

PTAL

[dmazzoni, 2015-04-21 22:39:02]

lgtm

https://chromiumcodereview.appspot.com/1100823004/diff/1/content/public/andro...
File
content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java
(right):

https://chromiumcodereview.appspot.com/1100823004/diff/1/content/public/andro...
content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java:2932:
public void onInitializeAccessibilityNodeInfo(AccessibilityNodeInfo info) {
You can delete this one. (AccessibilityNodeProvider completely replaces the need
for this.)

https://chromiumcodereview.appspot.com/1100823004/diff/1/content/public/andro...
content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java:2939:
public void onInitializeAccessibilityEvent(AccessibilityEvent event) {
You can delete this one. (AccessibilityNodeProvider completely replaces the need
for this.)

[aurimas (slooooooooow), 2015-04-21 23:23:02]

https://chromiumcodereview.appspot.com/1100823004/diff/1/content/public/andro...
File
content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java
(right):

https://chromiumcodereview.appspot.com/1100823004/diff/1/content/public/andro...
content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java:2932:
public void onInitializeAccessibilityNodeInfo(AccessibilityNodeInfo info) {
On 2015/04/21 22:39:02, dmazzoni wrote:
> You can delete this one. (AccessibilityNodeProvider completely replaces the
need
> for this.)

Done.

https://chromiumcodereview.appspot.com/1100823004/diff/1/content/public/andro...
content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java:2939:
public void onInitializeAccessibilityEvent(AccessibilityEvent event) {
On 2015/04/21 22:39:02, dmazzoni wrote:
> You can delete this one. (AccessibilityNodeProvider completely replaces the
need
> for this.)

Done.

[aurimas (slooooooooow), 2015-04-21 23:23:40]

aurimas@chromium.org changed reviewers:
+ boliu@chromium.org

[aurimas (slooooooooow), 2015-04-21 23:23:41]

+boliu for android_webview/

[David Trainor- moved to gerrit, 2015-04-22 01:30:06]

I forget, why did we have the LollipopAccessibilityInjector/why can we get rid
of it now?  Pending that question, this lgtm :).  Thanks Aurimas!

[dmazzoni, 2015-04-22 03:45:35]

I think LollipopAccessibilityInjector was just because some APIs were
deprecated and we switched to the new names in L to avoid deprecation
warnings. There was no logic difference.

On Tue, Apr 21, 2015 at 6:30 PM, <dtrainor@chromium.org> wrote:

> I forget, why did we have the LollipopAccessibilityInjector/why can we get
> rid
> of it now?  Pending that question, this lgtm :).  Thanks Aurimas!
>
>
> https://chromiumcodereview.appspot.com/1100823004/
>
> To unsubscribe from this group and stop receiving emails from it, send an
> email to chromium-reviews+unsubscribe@chromium.org.
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[boliu, 2015-04-22 06:33:21]

lgtm % one comment

\o/

https://chromiumcodereview.appspot.com/1100823004/diff/20001/android_webview/...
File android_webview/java/src/org/chromium/android_webview/AwContents.java
(left):

https://chromiumcodereview.appspot.com/1100823004/diff/20001/android_webview/...
android_webview/java/src/org/chromium/android_webview/AwContents.java:2247:
public void onInitializeAccessibilityNodeInfo(AccessibilityNodeInfo info) {
These are still called from WebViewChromium.java. Will need to fix both upstream
and downstream one (which shouldn't be done by hand, and can't be done with the
script right now...). No need to worry about the third_party copy.

I'd recommend leaving these no-op with a TODO for now.

[aurimas (slooooooooow), 2015-04-22 21:40:16]

https://chromiumcodereview.appspot.com/1100823004/diff/20001/android_webview/...
File android_webview/java/src/org/chromium/android_webview/AwContents.java
(left):

https://chromiumcodereview.appspot.com/1100823004/diff/20001/android_webview/...
android_webview/java/src/org/chromium/android_webview/AwContents.java:2247:
public void onInitializeAccessibilityNodeInfo(AccessibilityNodeInfo info) {
On 2015/04/22 at 06:33:21, boliu wrote:
> These are still called from WebViewChromium.java. Will need to fix both
upstream and downstream one (which shouldn't be done by hand, and can't be done
with the script right now...). No need to worry about the third_party copy.
> 
> I'd recommend leaving these no-op with a TODO for now.

Done

[aurimas (slooooooooow), 2015-04-22 21:45:55]

The CQ bit was checked by aurimas@chromium.org

[aurimas (slooooooooow), 2015-04-22 21:45:55]

The patchset sent to the CQ was uploaded after l-g-t-m from
dmazzoni@chromium.org, dtrainor@chromium.org, boliu@chromium.org
Link to the patchset: https://chromiumcodereview.appspot.com/1100823004/#ps40001
(title: "boliu's nits")

[commit-bot: I haz the power, 2015-04-22 21:52:29]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1100823004/40001

[aurimas (slooooooooow), 2015-04-22 23:07:59]

aurimas@chromium.org changed reviewers:
+ damienv@chromium.org

[aurimas (slooooooooow), 2015-04-22 23:08:18]

TBR damienv@chromium.org for trivial CastWindowAndroid change.

[aurimas (slooooooooow), 2015-04-22 23:08:20]

The CQ bit was checked by aurimas@chromium.org

[aurimas (slooooooooow), 2015-04-22 23:08:20]

The patchset sent to the CQ was uploaded after l-g-t-m from
dtrainor@chromium.org, dmazzoni@chromium.org, boliu@chromium.org
Link to the patchset: https://chromiumcodereview.appspot.com/1100823004/#ps60001
(title: "Update Chrome cast shell")

[commit-bot: I haz the power, 2015-04-22 23:10:29]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1100823004/60001

[commit-bot: I haz the power, 2015-04-23 01:14:49]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-04-23 01:15:55]

Patchset 4 (id:??) landed as
https://crrev.com/eeffed48a0847161775df9cec591446b80c83e54
Cr-Commit-Position: refs/heads/master@{#326419}