Implement TOGGLE_EXPANSION action for the ARC notifications.

Implement TOGGLE_EXPANSION action for the ARC notifications.

TOGGLE_EXPANSION action will be triggerred when a user triggers
DO_DEFAULT accessibility action on the notification.

I use the target of the mouse pressing event on
ArcCustomNotificationView to distinguish accessibility DO_DEFAULT
action and normal click on the notification.
This hack will be removed after making |AXActionData| be propagated to
|HandleAccessibleAction|.

BUG=703989
Review-Url: https://codereview.chromium.org/2767253005
Cr-Commit-Position: refs/heads/master@{#460087}
Committed: https://chromium.googlesource.com/chromium/src/+/b7af49bce7ecd2e79c8a54712d27bd45e27c2420

[yhanada, 2017-03-24 08:18:32]

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

[yhanada, 2017-03-24 08:18:32]

PTAL. Thanks!

[yhanada, 2017-03-24 08:18:40]

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

[commit-bot: I haz the power, 2017-03-24 08:19:02]

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 08:41:03]

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

[commit-bot: I haz the power, 2017-03-24 08:41:04]

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-24 08:53:48]

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

[commit-bot: I haz the power, 2017-03-24 08:53:58]

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 09:00:38]

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

[commit-bot: I haz the power, 2017-03-24 09:00:39]

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...)

[yhanada, 2017-03-24 10:06:37]

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:06:55]

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:49:21]

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

[commit-bot: I haz the power, 2017-03-24 10:49:22]

Dry run: This issue passed the CQ dry run.

[yoshiki, 2017-03-27 06:04:51]

lgtm with nit

https://codereview.chromium.org/2767253005/diff/20001/ui/message_center/views...
File ui/message_center/views/custom_notification_view.cc (right):

https://codereview.chromium.org/2767253005/diff/20001/ui/message_center/views...
ui/message_center/views/custom_notification_view.cc:162: if (contents_view_)
nit: please add brackets for multi-line if-block.

Or if (contents_view_ && contents_view->...()) is also good.

[yhanada, 2017-03-27 06:26:42]

https://codereview.chromium.org/2767253005/diff/20001/ui/message_center/views...
File ui/message_center/views/custom_notification_view.cc (right):

https://codereview.chromium.org/2767253005/diff/20001/ui/message_center/views...
ui/message_center/views/custom_notification_view.cc:162: if (contents_view_)
On 2017/03/27 06:04:51, yoshiki wrote:
> nit: please add brackets for multi-line if-block.
> 
> Or if (contents_view_ && contents_view->...()) is also good.

Done. Thanks!

[yhanada, 2017-03-27 06:26:51]

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

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

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 08:18:52]

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

[commit-bot: I haz the power, 2017-03-27 08:18:52]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_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:01:41]

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:02:12]

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 13:59:03]

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

[commit-bot: I haz the power, 2017-03-28 13:59:04]

Dry run: This issue passed the CQ dry run.

[yhanada, 2017-03-28 13:59:31]

The CQ bit was checked by yhanada@chromium.org

[yhanada, 2017-03-28 13:59:31]

The patchset sent to the CQ was uploaded after l-g-t-m from yoshiki@chromium.org
Link to the patchset: https://codereview.chromium.org/2767253005/#ps60001
(title: "rebase")

[commit-bot: I haz the power, 2017-03-28 13:59:46]

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 14:03:47]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1490709571174270,
"parent_rev": "7a8ec6e348f907a0f0f979b13608b312ed67d306", "commit_rev":
"b7af49bce7ecd2e79c8a54712d27bd45e27c2420"}

[commit-bot: I haz the power, 2017-03-28 14:04:29]

Description was changed from

==========
Implement TOGGLE_EXPANSION action for the ARC notifications.

TOGGLE_EXPANSION action will be triggerred when a user triggers
DO_DEFAULT accessibility action on the notification.

I use the target of the mouse pressing event on
ArcCustomNotificationView to distinguish accessibility DO_DEFAULT
action and normal click on the notification.
This hack will be removed after making |AXActionData| be propagated to
|HandleAccessibleAction|.

BUG=703989
==========

to

==========
Implement TOGGLE_EXPANSION action for the ARC notifications.

TOGGLE_EXPANSION action will be triggerred when a user triggers
DO_DEFAULT accessibility action on the notification.

I use the target of the mouse pressing event on
ArcCustomNotificationView to distinguish accessibility DO_DEFAULT
action and normal click on the notification.
This hack will be removed after making |AXActionData| be propagated to
|HandleAccessibleAction|.

BUG=703989

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

[commit-bot: I haz the power, 2017-03-28 14:04:31]

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