Hide cursor upon lock animation

Hide cursor upon lock animation

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

[Denis Kuznetsov (DE-MUC), 2013-04-08 18:12:18]

Please use LGTM&CQ.

[Daniel Erat, 2013-04-08 18:39:49]

https://chromiumcodereview.appspot.com/13637009/diff/2001/ash/wm/session_stat...
File ash/wm/session_state_controller_impl2.cc (right):

https://chromiumcodereview.appspot.com/13637009/diff/2001/ash/wm/session_stat...
ash/wm/session_state_controller_impl2.cc:267:
//ash::Shell::GetInstance()->cursor_manager()->ShowCursor();
what did you plan to do with this?  i agree that the cursor should be re-shown
when the animation is canceled -- do things work when you uncomment this code?

[Denis Kuznetsov (DE-MUC), 2013-04-08 18:44:49]

https://chromiumcodereview.appspot.com/13637009/diff/2001/ash/wm/session_stat...
File ash/wm/session_state_controller_impl2.cc (right):

https://chromiumcodereview.appspot.com/13637009/diff/2001/ash/wm/session_stat...
ash/wm/session_state_controller_impl2.cc:267:
//ash::Shell::GetInstance()->cursor_manager()->ShowCursor();
On 2013/04/08 18:39:49, Daniel Erat wrote:
> what did you plan to do with this?  i agree that the cursor should be re-shown
> when the animation is canceled -- do things work when you uncomment this code?
Sorry, that was the part of experiment and was not removed correctly.
Everything works fine without this ShowCursor due to
set_cursor_hidden_by_filter.

[Daniel Erat, 2013-04-08 19:28:18]

lgtm

https://chromiumcodereview.appspot.com/13637009/diff/2001/ash/wm/session_stat...
File ash/wm/session_state_controller_impl2.cc (right):

https://chromiumcodereview.appspot.com/13637009/diff/2001/ash/wm/session_stat...
ash/wm/session_state_controller_impl2.cc:267:
//ash::Shell::GetInstance()->cursor_manager()->ShowCursor();
On 2013/04/08 18:44:49, Denis Kuznetsov wrote:
> On 2013/04/08 18:39:49, Daniel Erat wrote:
> > what did you plan to do with this?  i agree that the cursor should be
re-shown
> > when the animation is canceled -- do things work when you uncomment this
code?
> Sorry, that was the part of experiment and was not removed correctly.
> Everything works fine without this ShowCursor due to
> set_cursor_hidden_by_filter.

I meant that the cursor should maybe be shown even before the user moves the
mouse.  I don't have strong feelings about this, though, so LGTM.

[commit-bot: I haz the power, 2013-04-08 19:28:36]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/antrim@chromium.org/13637009/5004

[commit-bot: I haz the power, 2013-04-09 01:40:10]

Change committed as 192979