Process all global shortcuts for Ash after InputMethodEventFilter

Process all global shortcuts for Ash after InputMethodEventFilter.

This CL is for crbug.com/123856 (Aura shell applies accelerators without giving apps the option of handling key events). Currently, shortcuts for IME/layout switching (Shift+Alt, Ctrl+space, and so on) are processed before InputMethodEventFilter, and other Ash global shortcuts are processed after the filter. This is for ensuring that the IME shortcuts always work even if an IME (or an IME extension - http://dev.chromium.org/developers/design-documents/extensions/proposed-changes/apis-under-development/input-method-editor) tries to consume the shortcuts. However, this strategy is not compatible with crbug.com/123856. In order to fix crbug.com/123856, we sometimes have to process an Ash global shortcut key (including IME ones) after the key is passed to a web page.

This CL modifies accelerator_table.cc and shell.cc so that they process IME shortcuts after InputMethodEventFilter, and also simplifies ash/accelerators/ and ui/base/accelerators/accelerator_manager.cc by removing code for processing pre-IME shortcuts.

BUG=123856
TEST=ran aura_shell_unittests and ui_unittests

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=133642

[mazda, 2012-04-23 16:46:16]

LGTM with a nit

http://codereview.chromium.org/10155015/diff/4001/ash/accelerators/accelerato...
File ash/accelerators/accelerator_table.cc (right):

http://codereview.chromium.org/10155015/diff/4001/ash/accelerators/accelerato...
ash/accelerators/accelerator_table.cc:96: // EventType, KeyboardCode, shift,
control, alt, AcceleratorAction
nit: Please modify the comment here as well.

[Ben Goodger (Google), 2012-04-23 16:48:22]

LGTM2

[Yusuke Sato, 2012-04-24 06:07:31]

http://codereview.chromium.org/10155015/diff/4001/ash/accelerators/accelerato...
File ash/accelerators/accelerator_table.cc (right):

http://codereview.chromium.org/10155015/diff/4001/ash/accelerators/accelerato...
ash/accelerators/accelerator_table.cc:96: // EventType, KeyboardCode, shift,
control, alt, AcceleratorAction
On 2012/04/23 16:46:16, mazda wrote:
> nit: Please modify the comment here as well.

Done.

[commit-bot: I haz the power, 2012-04-24 06:14:33]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/yusukes@chromium.org/10155015/12001