Allow Control+Shift+Q on lock screen.

Allow Control+Shift+Q on lock screen.

Usually, when the screen is locked and the status area at the bottom right corner of the screen is not focused, Control+Shift+Q is handled by chrome/browser/chromeos/login/webui_login_view.cc. But when the status area is focused, Control+Shift+Q is handled by Ash, accelerator_controller.cc.

BUG=120953
TEST=aura_shell_unittests + manual: focus the area and press C+S+Q on both login and lock screens


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=138509

[Daniel Erat, 2012-05-23 13:45:20]

lgtm

http://codereview.chromium.org/10388230/diff/3001/ash/accelerators/accelerato...
File ash/accelerators/accelerator_table.h (right):

http://codereview.chromium.org/10388230/diff/3001/ash/accelerators/accelerato...
ash/accelerators/accelerator_table.h:93: // Actions allowed while user is
locked.
nit: update this to read "Actions allowed while screen is locked (in addition to
kActionsAllowedAtLoginOrLockScreen)."

[Yusuke Sato, 2012-05-23 15:36:56]

http://codereview.chromium.org/10388230/diff/3001/ash/accelerators/accelerato...
File ash/accelerators/accelerator_table.h (right):

http://codereview.chromium.org/10388230/diff/3001/ash/accelerators/accelerato...
ash/accelerators/accelerator_table.h:93: // Actions allowed while user is
locked.
On 2012/05/23 13:45:20, Daniel Erat wrote:
> nit: update this to read "Actions allowed while screen is locked (in addition
to
> kActionsAllowedAtLoginOrLockScreen)."

Thanks, fixed.

[commit-bot: I haz the power, 2012-05-23 15:37:24]

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

[commit-bot: I haz the power, 2012-05-23 17:17:05]

Change committed as 138509