Converts accNavigate over to the AXPlatformNode code.

Converts accNavigate over to the AXPlatformNode code.

This CL switches accNavigate completely over to the AXPlatformNode code.
It also moves the table handling code from BrowserAccessibility to
AXPlatformNodeBase.

BUG=703369
Review-Url: https://codereview.chromium.org/2964313002
Cr-Commit-Position: refs/heads/master@{#485887}
Committed: https://chromium.googlesource.com/chromium/src/+/9344ce7d3f2fe16e08af9a0902423d5d3b27a34c

[dougt, 2017-06-30 23:13:56]

The CQ bit was checked by dougt@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-30 23:14:11]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-06-30 23:16:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-30 23:16:58]

Dry run: Try jobs failed on following builders:
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[dougt, 2017-07-01 02:12:00]

The CQ bit was checked by dougt@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-07-01 02:12:08]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[dougt, 2017-07-01 04:17:25]

dougt@chromium.org changed reviewers:
+ dmazzoni@chromium.org

[dougt, 2017-07-01 04:17:26]

dmazzoni, ptal.

[commit-bot: I haz the power, 2017-07-01 05:03:40]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-07-01 05:03:41]

Dry run: This issue passed the CQ dry run.

[dmazzoni, 2017-07-06 23:07:48]

lgtm

Might be worth manually testing this with JAWS and
NVDA. In browse mode you should be able to move
between table cells with Ctrl+Alt+Arrows.

https://codereview.chromium.org/2964313002/diff/60001/ui/accessibility/platfo...
File ui/accessibility/platform/ax_platform_node_win.cc (right):

https://codereview.chromium.org/2964313002/diff/60001/ui/accessibility/platfo...
ui/accessibility/platform/ax_platform_node_win.cc:391: // This direction are not
implemented except in tables.
are not -> is not, throughout

[dougt, 2017-07-11 23:21:55]

Tested with both nvda and jaws. Was able to navigate tables without issue.

https://codereview.chromium.org/2964313002/diff/60001/ui/accessibility/platfo...
File ui/accessibility/platform/ax_platform_node_win.cc (right):

https://codereview.chromium.org/2964313002/diff/60001/ui/accessibility/platfo...
ui/accessibility/platform/ax_platform_node_win.cc:391: // This direction are not
implemented except in tables.
On 2017/07/06 23:07:48, dmazzoni_ooo_until_july_17 wrote:
> are not -> is not, throughout

Done.

[dougt, 2017-07-12 05:12:18]

The CQ bit was checked by dougt@chromium.org

[dougt, 2017-07-12 05:12:18]

The patchset sent to the CQ was uploaded after l-g-t-m from
dmazzoni@chromium.org
Link to the patchset: https://codereview.chromium.org/2964313002/#ps80001
(title: "are -> is")

[commit-bot: I haz the power, 2017-07-12 05:12:25]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-07-12 07:17:16]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1499836338101640,
"parent_rev": "108fc33c12e6ba7001b0f530e5d1bf660079da6a", "commit_rev":
"9344ce7d3f2fe16e08af9a0902423d5d3b27a34c"}

[commit-bot: I haz the power, 2017-07-12 07:17:29]

Description was changed from

==========
Converts accNavigate over to the AXPlatformNode code.

This CL switches accNavigate completely over to the AXPlatformNode code.
It also moves the table handling code from BrowserAccessibility to
AXPlatformNodeBase.

BUG=703369
==========

to

==========
Converts accNavigate over to the AXPlatformNode code.

This CL switches accNavigate completely over to the AXPlatformNode code.
It also moves the table handling code from BrowserAccessibility to
AXPlatformNodeBase.

BUG=703369

Review-Url: https://codereview.chromium.org/2964313002
Cr-Commit-Position: refs/heads/master@{#485887}
Committed:
https://chromium.googlesource.com/chromium/src/+/9344ce7d3f2fe16e08af9a090242...
==========

[commit-bot: I haz the power, 2017-07-12 07:17:31]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/9344ce7d3f2fe16e08af9a090242...