SimpleMenuModel and ButtonMenuItemModel implement AcceleratorProvider.

SimpleMenuModel and ButtonMenuItemModel implement AcceleratorProvider.

(Actually applies to their Delegate classes, not those classes
themselves.) These classes were providing the full AcceleratorProvider
interface, so may as well officially implement it. This cleans up some
awkward subclasses that inherit both AcceleratorProvider and
SimpleMenuModel::Delegate and override the same method in both.

BUG=627344
Committed: https://crrev.com/164de0bd6781df7e12d2845ba24689ef74a6183e
Cr-Commit-Position: refs/heads/master@{#409138}

[Matt Giuca, 2016-08-01 05:19:31]

Patchset #4 (id:60001) has been deleted

[Matt Giuca, 2016-08-01 05:24:22]

mgiuca@chromium.org changed reviewers:
+ sky@chromium.org

[Matt Giuca, 2016-08-01 05:24:23]

Scott: PTAL, thanks. (Final piece of https://crbug.com/627344 refactor.)

[sky, 2016-08-01 15:03:29]

LGTM

[Matt Giuca, 2016-08-02 01:18:11]

The CQ bit was checked by mgiuca@chromium.org

[commit-bot: I haz the power, 2016-08-02 01:18:29]

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 02:07:18]

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

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

Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[Matt Giuca, 2016-08-02 03:24:29]

Description was changed from

==========
SimpleMenuModel and ButtonMenuItemModel implement AcceleratorProvider.

These classes were providing the full AcceleratorProvider interface, so
may as well officially implement it. This cleans up some awkward
subclasses that inherit both AcceleratorProvider and SimpleMenuModel and
override the same method in both.

BUG=627344
==========

to

==========
SimpleMenuModel and ButtonMenuItemModel implement AcceleratorProvider.

(Actually applies to their Delegate classes, not those classes
themselves.) These classes were providing the full AcceleratorProvider
interface, so may as well officially implement it. This cleans up some
awkward subclasses that inherit both AcceleratorProvider and
SimpleMenuModel::Delegate and override the same method in both.

BUG=627344
==========

[Matt Giuca, 2016-08-02 03:28:31]

Patchset #4 (id:80001) has been deleted

[Matt Giuca, 2016-08-02 03:35:50]

The CQ bit was checked by mgiuca@chromium.org

[Matt Giuca, 2016-08-02 03:35:51]

The patchset sent to the CQ was uploaded after l-g-t-m from sky@chromium.org
Link to the patchset: https://codereview.chromium.org/2138353002/#ps100001
(title: "Fix compile on Windows.")

[commit-bot: I haz the power, 2016-08-02 03:36: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-02 04:35:52]

Committed patchset #4 (id:100001)

[commit-bot: I haz the power, 2016-08-02 04:37:28]

Description was changed from

==========
SimpleMenuModel and ButtonMenuItemModel implement AcceleratorProvider.

(Actually applies to their Delegate classes, not those classes
themselves.) These classes were providing the full AcceleratorProvider
interface, so may as well officially implement it. This cleans up some
awkward subclasses that inherit both AcceleratorProvider and
SimpleMenuModel::Delegate and override the same method in both.

BUG=627344
==========

to

==========
SimpleMenuModel and ButtonMenuItemModel implement AcceleratorProvider.

(Actually applies to their Delegate classes, not those classes
themselves.) These classes were providing the full AcceleratorProvider
interface, so may as well officially implement it. This cleans up some
awkward subclasses that inherit both AcceleratorProvider and
SimpleMenuModel::Delegate and override the same method in both.

BUG=627344

Committed: https://crrev.com/164de0bd6781df7e12d2845ba24689ef74a6183e
Cr-Commit-Position: refs/heads/master@{#409138}
==========

[commit-bot: I haz the power, 2016-08-02 04:37:29]

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