Fix animation distance.

ui/keyboard/keyboard_controller.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/keyboard/keyboard_controller.h"
 
 #include <set>
 
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 352 matching lines @@
       container_animator,
       base::Bind(&KeyboardController::HideAnimationFinished,
                  base::Unretained(this))));
   container_animator->AddObserver(animation_observer_.get());
 
   ui::ScopedLayerAnimationSettings settings(container_animator);
   settings.SetTweenType(gfx::Tween::FAST_OUT_LINEAR_IN);
   settings.SetTransitionDuration(
       base::TimeDelta::FromMilliseconds(kHideAnimationDurationMs));
   gfx::Transform transform;
-  transform.Translate(0, proxy_->GetKeyboardWindow()->bounds().height());
+  transform.Translate(0, keyboard::kAnimationDistance);

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

nit: I believe "keyboard" namespace is not necessary as it is already in
keyboard namespace.

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

Done.

   container_->SetTransform(transform);
   container_->layer()->SetOpacity(kAnimationStartOrAfterHideOpacity);
 }
 
 void KeyboardController::AddObserver(KeyboardControllerObserver* observer) {
   observer_list_.AddObserver(observer);
 }
 
 void KeyboardController::RemoveObserver(KeyboardControllerObserver* observer) {
   observer_list_.RemoveObserver(observer);
@@ skipping 132 matching lines @@
       !container_->layer()->GetAnimator()->is_animating())
     return;
 
   ToggleTouchEventLogging(false);
   ui::LayerAnimator* container_animator = container_->layer()->GetAnimator();
 
   // If the container is not animating, makes sure the position and opacity
   // are at begin states for animation.
   if (!container_animator->is_animating()) {
     gfx::Transform transform;
-    transform.Translate(0, proxy_->GetKeyboardWindow()->bounds().height());
+    transform.Translate(0, keyboard::kAnimationDistance);

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

nit: ditto

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

Done.

     container_->SetTransform(transform);
     container_->layer()->SetOpacity(kAnimationStartOrAfterHideOpacity);
   }
 
   container_animator->set_preemption_strategy(
       ui::LayerAnimator::IMMEDIATELY_ANIMATE_TO_NEW_TARGET);
   animation_observer_.reset(new CallbackAnimationObserver(
       container_animator,
       base::Bind(&KeyboardController::ShowAnimationFinished,
                  base::Unretained(this))));
@@ skipping 42 matching lines @@
   proxy_->HideKeyboardContainer(container_.get());
 }
 
 void KeyboardController::AddBoundsChangedObserver(aura::Window* window) {
   aura::Window* target_window = GetFrameWindow(window);
   if (target_window)
     window_bounds_observer_->AddObservedWindow(target_window);
 }
 
 }  // namespace keyboard