Try |View::HandleAccessibleAction| before falling back to fixed actions.

Try |View::HandleAccessibleAction| before falling back to fixed actions.

Currently automation API always do the fixed action for each
accessible action. For example, clicking on the center of the target
view is always triggered when AX_ACTION_DO_DEFAULT is passed to
automation API.

This CL makes automation API try to use |View::HandleAccessibleAction|
first. This change enables us to implement custom action on accessible
actions.

BUG=705350
Review-Url: https://codereview.chromium.org/2767143007
Cr-Commit-Position: refs/heads/master@{#460255}
Committed: https://chromium.googlesource.com/chromium/src/+/e80b960e27af6495ff3dd79cbfcede18d48da85a

[yhanada, 2017-03-24 10:05:34]

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

[commit-bot: I haz the power, 2017-03-24 10:05:51]

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-03-24 10:18:51]

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

[commit-bot: I haz the power, 2017-03-24 10:18:52]

Dry run: Try jobs failed on following builders:
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[yhanada, 2017-03-27 05:52:43]

Description was changed from

==========
Try |View::HandleAccessibleAction| before falling back to fixed actions.

Currently automation API always do the fixed action for each
accessible action. For example, clicking on the center of the target
view is always triggered when AX_ACTION_DO_DEFAULT is passed to
automation API.

This CL makes automation API try to use |View::HandleAccessibleAction|
first. This change enables us to implement custom action on accessible
actions.

BUG=TBF
==========

to

==========
Try |View::HandleAccessibleAction| before falling back to fixed actions.

Currently automation API always do the fixed action for each
accessible action. For example, clicking on the center of the target
view is always triggered when AX_ACTION_DO_DEFAULT is passed to
automation API.

This CL makes automation API try to use |View::HandleAccessibleAction|
first. This change enables us to implement custom action on accessible
actions.

BUG=705350
==========

[yhanada, 2017-03-27 05:55:43]

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

[commit-bot: I haz the power, 2017-03-27 05:56:00]

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-03-27 06:05:12]

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

[commit-bot: I haz the power, 2017-03-27 06:05:14]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[yhanada, 2017-03-28 08:26:55]

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

[commit-bot: I haz the power, 2017-03-28 08:27:04]

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-03-28 08:36:10]

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

[commit-bot: I haz the power, 2017-03-28 08:36:11]

Dry run: Try jobs failed on following builders:
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[yhanada, 2017-03-28 13:06:57]

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

[commit-bot: I haz the power, 2017-03-28 13:07:15]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[yhanada, 2017-03-28 14:06:41]

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

[yhanada, 2017-03-28 14:06:42]

PTAL. Thanks!

[commit-bot: I haz the power, 2017-03-28 14:17:40]

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

[commit-bot: I haz the power, 2017-03-28 14:17:41]

Dry run: This issue passed the CQ dry run.

[dmazzoni, 2017-03-28 15:02:24]

lgtm

Thanks!

This is fine as-is, I can follow this up with a change that
pushes more of the implementation directly into View,
but this is a great start and it unblocks the change
you're making.

[yhanada, 2017-03-28 22:44:40]

Thank you for reviewing! I submit this change as-is.

[yhanada, 2017-03-28 22:45:03]

The CQ bit was checked by yhanada@chromium.org

[yhanada, 2017-03-28 22:45:12]

The CQ bit was unchecked by yhanada@chromium.org

[yhanada, 2017-03-28 22:50:06]

yhanada@chromium.org changed reviewers:
+ msw@chromium.org

[yhanada, 2017-03-28 22:50:06]

Hi msw@, could you review ui/views/BUILD.gn change as the OWNER?

[msw, 2017-03-28 23:58:03]

ui/views/BUILD.gn lgtm

[yhanada, 2017-03-29 00:24:50]

Thank you!

[yhanada, 2017-03-29 00:24:55]

The CQ bit was checked by yhanada@chromium.org

[commit-bot: I haz the power, 2017-03-29 00:26:07]

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-03-29 01:00:56]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1490747095869720,
"parent_rev": "08cae037b8ee8f225471e4f6b64230f20d1bd5c1", "commit_rev":
"e80b960e27af6495ff3dd79cbfcede18d48da85a"}

[commit-bot: I haz the power, 2017-03-29 01:04:52]

Description was changed from

==========
Try |View::HandleAccessibleAction| before falling back to fixed actions.

Currently automation API always do the fixed action for each
accessible action. For example, clicking on the center of the target
view is always triggered when AX_ACTION_DO_DEFAULT is passed to
automation API.

This CL makes automation API try to use |View::HandleAccessibleAction|
first. This change enables us to implement custom action on accessible
actions.

BUG=705350
==========

to

==========
Try |View::HandleAccessibleAction| before falling back to fixed actions.

Currently automation API always do the fixed action for each
accessible action. For example, clicking on the center of the target
view is always triggered when AX_ACTION_DO_DEFAULT is passed to
automation API.

This CL makes automation API try to use |View::HandleAccessibleAction|
first. This change enables us to implement custom action on accessible
actions.

BUG=705350

Review-Url: https://codereview.chromium.org/2767143007
Cr-Commit-Position: refs/heads/master@{#460255}
Committed:
https://chromium.googlesource.com/chromium/src/+/e80b960e27af6495ff3dd79cbfce...
==========

[commit-bot: I haz the power, 2017-03-29 01:04:56]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/e80b960e27af6495ff3dd79cbfce...