Fix animation distance.

Fix animation distance.

BUG=355572
Committed: https://crrev.com/3293cbdadb5d001832c7d6ce7918a5a90b8ba8b6
Cr-Commit-Position: refs/heads/master@{#295075}

[rsadam, 2014-09-15 17:23:36]

rsadam@chromium.org changed reviewers:
+ bshe@chromium.org

[bshe, 2014-09-15 20:39:55]

On 2014/09/15 17:23:37, rsadam wrote:

you probably need to update a few keyboard_unittests

[rsadam, 2014-09-16 14:32:29]

Done!

[bshe, 2014-09-16 14:43:38]

lgtm with nits

https://codereview.chromium.org/544173006/diff/40001/ui/keyboard/keyboard_con...
File ui/keyboard/keyboard_controller.cc (right):

https://codereview.chromium.org/544173006/diff/40001/ui/keyboard/keyboard_con...
ui/keyboard/keyboard_controller.cc:373: transform.Translate(0,
keyboard::kAnimationDistance);
nit: I believe "keyboard" namespace is not necessary as it is already in
keyboard namespace.

https://codereview.chromium.org/544173006/diff/40001/ui/keyboard/keyboard_con...
ui/keyboard/keyboard_controller.cc:526: transform.Translate(0,
keyboard::kAnimationDistance);
nit: ditto

[rsadam, 2014-09-16 14:46:58]

https://codereview.chromium.org/544173006/diff/40001/ui/keyboard/keyboard_con...
File ui/keyboard/keyboard_controller.cc (right):

https://codereview.chromium.org/544173006/diff/40001/ui/keyboard/keyboard_con...
ui/keyboard/keyboard_controller.cc:373: transform.Translate(0,
keyboard::kAnimationDistance);
On 2014/09/16 14:43:38, bshe wrote:
> nit: I believe "keyboard" namespace is not necessary as it is already in
> keyboard namespace.

Done.

https://codereview.chromium.org/544173006/diff/40001/ui/keyboard/keyboard_con...
ui/keyboard/keyboard_controller.cc:526: transform.Translate(0,
keyboard::kAnimationDistance);
On 2014/09/16 14:43:38, bshe wrote:
> nit: ditto

Done.

[rsadam, 2014-09-16 14:47:04]

The CQ bit was checked by rsadam@chromium.org

[commit-bot: I haz the power, 2014-09-16 14:47:32]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patchset/544173006/60001

[commit-bot: I haz the power, 2014-09-16 15:45:17]

Committed patchset #4 (id:60001) as e30e2ab7f3332e4ba333454fa0e11d997b3498e1

[commit-bot: I haz the power, 2014-09-16 15:46:41]

Patchset 4 (id:??) landed as
https://crrev.com/3293cbdadb5d001832c7d6ce7918a5a90b8ba8b6
Cr-Commit-Position: refs/heads/master@{#295075}