AcceleratorController::AcceleratorPressed() should return true whenever some action is taken

AcceleratorController::AcceleratorPressed() should return true whenever some action is taken.

Currently some actions do not follow the description of ui::AcceleratorTarget::AcceleratorPressed() in ui/base/accelerators/accelerator.h (see below).  Though the violations don't cause any user visible issues at this moment, I believe they should be fixed.

  // Should return true if the accelerator was processed.
  virtual bool AcceleratorPressed(const Accelerator& accelerator) = 0;

This change is also necessary for fixing 123856.

BUG=123856
TEST=aura_shell_unittests

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

[Daniel Erat, 2012-05-08 13:52:22]

lgtm

[mazda, 2012-05-08 22:32:08]

lgtm

[commit-bot: I haz the power, 2012-05-10 06:23:43]

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

[commit-bot: I haz the power, 2012-05-10 07:05:14]

Try job failure for 10388023-5001 (retry) on mac_rel for step "browser_tests"
(clobber build).
It's a second try, previously, step "compile" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2012-05-10 07:09:35]

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

[commit-bot: I haz the power, 2012-05-10 07:27:23]

Try job failure for 10388023-5001 (retry) on android for steps "Compile, build".
It's a second try, previously, steps "Compile, build" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=android&nu...