ash: Remove old lock animation implementation.

ash: Remove old lock animation implementation.

This deletes the SessionStateControllerImpl class (formerly
used when the --ash-disable-new-lock-animations flag, which
I'm also removing, was set) and moves
LockStateControllerImpl2 into LockStateController (which was
formerly an interface).

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

[Daniel Erat, 2013-09-27 23:20:17]

https://codereview.chromium.org/24980006/diff/3001/ash/wm/power_button_contro...
File ash/wm/power_button_controller_unittest.cc (left):

https://codereview.chromium.org/24980006/diff/3001/ash/wm/power_button_contro...
ash/wm/power_button_controller_unittest.cc:1: // Copyright (c) 2012 The Chromium
Authors. All rights reserved.
These tests were setting the old disable flag and the PowerButtonController
class is tiny now, so I don't think we're getting any useful test coverage from
it anymore.

[Nikita (slow), 2013-09-30 09:56:22]

lgtm

See also https://code.google.com/p/chromium/issues/detail?id=162645 that is
about re-enabling test coverage for new animations which appears to be blocked.
Denis should know better that status.

[Denis Kuznetsov (DE-MUC), 2013-09-30 12:45:51]

lgtm

[commit-bot: I haz the power, 2013-09-30 15:59:38]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/derat@chromium.org/24980006/11001

[commit-bot: I haz the power, 2013-09-30 20:15:51]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/derat@chromium.org/24980006/34001

[commit-bot: I haz the power, 2013-10-01 04:22:47]

Retried try job too often on mac_rel for step(s) content_browsertests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2013-10-01 12:40:53]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/derat@chromium.org/24980006/34001

[commit-bot: I haz the power, 2013-10-01 21:08:36]

Change committed as 226298