[Extensions UI Cocoa] Make disabled actions respond to the keyboard

[Extensions UI Cocoa] Make disabled actions respond to the keyboard

There was a bug where an otherwise-disabled action wouldn't respond to key
presses. Fix this, and add a test.

BUG=592825
Committed: https://crrev.com/98f7b6d97ccaa3c469319a0e1bbc051885847d0b
Cr-Commit-Position: refs/heads/master@{#380659}

[Devlin, 2016-03-10 01:10:28]

Patchset #1 (id:1) has been deleted

[Devlin, 2016-03-10 01:10:47]

rdevlin.cronin@chromium.org changed reviewers:
+ avi@chromium.org, finnur@chromium.org

[Devlin, 2016-03-10 01:10:48]

Finnur and Avi, mind taking a look?

[Avi (use Gerrit), 2016-03-10 04:06:01]

lgtm

[Finnur, 2016-03-10 13:35:43]

LGTM

[Devlin, 2016-03-10 23:09:41]

The CQ bit was checked by rdevlin.cronin@chromium.org

[Devlin, 2016-03-10 23:09:42]

The patchset sent to the CQ was uploaded after l-g-t-m from finnur@chromium.org,
avi@chromium.org
Link to the patchset: https://codereview.chromium.org/1784533003/#ps40001
(title: "Linux compile fix")

[commit-bot: I haz the power, 2016-03-10 23:10:34]

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

[commit-bot: I haz the power, 2016-03-10 23:45:30]

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

[commit-bot: I haz the power, 2016-03-10 23:45:31]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[Devlin, 2016-03-10 23:47:54]

The CQ bit was checked by rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2016-03-10 23:48:54]

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

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

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

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

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

[Devlin, 2016-03-11 01:31:24]

The CQ bit was checked by rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2016-03-11 01:32:46]

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

[commit-bot: I haz the power, 2016-03-11 02:03:08]

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

[commit-bot: I haz the power, 2016-03-11 02:03:09]

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

[Devlin, 2016-03-11 16:20:36]

The CQ bit was checked by rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2016-03-11 16:20:55]

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

[commit-bot: I haz the power, 2016-03-11 17:21:16]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-03-11 17:23:06]

Description was changed from

==========
[Extensions UI Cocoa] Make disabled actions respond to the keyboard

There was a bug where an otherwise-disabled action wouldn't respond to key
presses. Fix this, and add a test.

BUG=592825
==========

to

==========
[Extensions UI Cocoa] Make disabled actions respond to the keyboard

There was a bug where an otherwise-disabled action wouldn't respond to key
presses. Fix this, and add a test.

BUG=592825

Committed: https://crrev.com/98f7b6d97ccaa3c469319a0e1bbc051885847d0b
Cr-Commit-Position: refs/heads/master@{#380659}
==========

[commit-bot: I haz the power, 2016-03-11 17:23:07]

Patchset 2 (id:??) landed as
https://crrev.com/98f7b6d97ccaa3c469319a0e1bbc051885847d0b
Cr-Commit-Position: refs/heads/master@{#380659}