Get AcceleratorControllerTest working with mash

Get AcceleratorControllerTest working with mash

A couple of tests have been commented or disabled because of current
lacking functionality in mash. Bugs are filed for all the cases.

BUG=612331, 632208, 632209, 630683, 632211, 632180, 632189, 632192, 632200, 632201
TEST=AcceleratorControllerTest* in mash_unittests
R=sadrul@chromium.org

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

[sky, 2016-08-02 17:23:39]

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

[commit-bot: I haz the power, 2016-08-02 17:24:22]

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, 2016-08-02 17:27:22]

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

[commit-bot: I haz the power, 2016-08-02 17:27:23]

Dry run: Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  mac_chromium_compile_dbg_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[sky, 2016-08-02 17:48:39]

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

[commit-bot: I haz the power, 2016-08-02 17:49:43]

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, 2016-08-02 19:26:01]

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

[commit-bot: I haz the power, 2016-08-02 19:26:02]

Dry run: This issue passed the CQ dry run.

[sadrul, 2016-08-02 22:39:51]

lgtm

https://codereview.chromium.org/2203003003/diff/40001/ash/mus/BUILD.gn
File ash/mus/BUILD.gn (right):

https://codereview.chromium.org/2203003003/diff/40001/ash/mus/BUILD.gn#newcod...
ash/mus/BUILD.gn:231: "//ui/message_center:message_center",
Just ui/message_center?

https://codereview.chromium.org/2203003003/diff/40001/ash/mus/accelerators/ac...
File ash/mus/accelerators/accelerator_controller_unittest.cc (right):

https://codereview.chromium.org/2203003003/diff/40001/ash/mus/accelerators/ac...
ash/mus/accelerators/accelerator_controller_unittest.cc:661: //   test, but no
point until we decide on the first one.
crbug.com/615033 for the EventGenerator issue.

[sky, 2016-08-02 22:50:38]

https://codereview.chromium.org/2203003003/diff/40001/ash/mus/BUILD.gn
File ash/mus/BUILD.gn (right):

https://codereview.chromium.org/2203003003/diff/40001/ash/mus/BUILD.gn#newcod...
ash/mus/BUILD.gn:231: "//ui/message_center:message_center",
On 2016/08/02 22:39:50, sadrul wrote:
> Just ui/message_center?

Done.

https://codereview.chromium.org/2203003003/diff/40001/ash/mus/accelerators/ac...
File ash/mus/accelerators/accelerator_controller_unittest.cc (right):

https://codereview.chromium.org/2203003003/diff/40001/ash/mus/accelerators/ac...
ash/mus/accelerators/accelerator_controller_unittest.cc:661: //   test, but no
point until we decide on the first one.
On 2016/08/02 22:39:51, sadrul wrote:
> crbug.com/615033 for the EventGenerator issue.

Done.

[sky, 2016-08-02 22:50:42]

The CQ bit was checked by sky@chromium.org

[sky, 2016-08-02 22:50:43]

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

[commit-bot: I haz the power, 2016-08-02 22:51:33]

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, 2016-08-02 23:07:41]

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

[commit-bot: I haz the power, 2016-08-02 23:07:42]

Try jobs failed on following builders:
  chromeos_daisy_chromium_compile_only_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_daisy_...)

[sky, 2016-08-03 00:02:22]

The CQ bit was checked by sky@chromium.org

[commit-bot: I haz the power, 2016-08-03 00:03:06]

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, 2016-08-03 00:15:11]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-08-03 00:18:37]

Description was changed from

==========
Get AcceleratorControllerTest working with mash

A couple of tests have been commented or disabled because of current
lacking functionality in mash. Bugs are filed for all the cases.

BUG=612331,632208,632209,630683,632211,632180,632189,632192,632200,632201
TEST=AcceleratorControllerTest* in mash_unittests
R=sadrul@chromium.org
==========

to

==========
Get AcceleratorControllerTest working with mash

A couple of tests have been commented or disabled because of current
lacking functionality in mash. Bugs are filed for all the cases.

BUG=612331,632208,632209,630683,632211,632180,632189,632192,632200,632201
TEST=AcceleratorControllerTest* in mash_unittests
R=sadrul@chromium.org

Committed: https://crrev.com/aec2890d8c57c6b2b9b149b3d9eac2db5a4f014e
Cr-Commit-Position: refs/heads/master@{#409393}
==========

[commit-bot: I haz the power, 2016-08-03 00:18:38]

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