Remove redundant SetBounds calls to the keyboard window.

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 25 matching lines @@
 
 namespace {
 
 const int kHideKeyboardDelayMs = 100;
 
 // The virtual keyboard show/hide animation duration.
 const int kShowAnimationDurationMs = 350;
 const int kHideAnimationDurationMs = 100;
 
 // The opacity of virtual keyboard container when show animation starts or
-// hide animation finishes.
-// TODO(rsadam@): Investigate why setting this to zero crashes.
+// hide animation finishes. This cannot be zero because we call Show() on the
+// keyboard window before setting the opacity back to 1.0. Since windows are not
+// allowed to be shown with zero opacity, we always animate to 0.01 instead.
 const float kAnimationStartOrAfterHideOpacity = 0.01f;
 
 // Event targeter for the keyboard container.
 class KeyboardContainerTargeter : public wm::MaskedWindowTargeter {
  public:
   KeyboardContainerTargeter(aura::Window* container,
                             keyboard::KeyboardControllerProxy* proxy)
       : wm::MaskedWindowTargeter(container),
         proxy_(proxy) {
   }
@@ skipping 519 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