Remove redundant SetBounds calls to the keyboard window.

ui/keyboard/keyboard_controller_proxy.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_proxy.h"
 
 #include "base/command_line.h"
 #include "base/values.h"
 #include "content/public/browser/site_instance.h"
 #include "content/public/browser/web_contents.h"
@@ skipping 42 matching lines @@
                             const gfx::Rect& pos) OVERRIDE {
     aura::Window* keyboard = proxy_->GetKeyboardWindow();
     // keyboard window must have been added to keyboard container window at this
     // point. Otherwise, wrong keyboard bounds is used and may cause problem as
     // described in crbug.com/367788.
     DCHECK(keyboard->parent());
     gfx::Rect bounds = keyboard->bounds();
     int new_height = pos.height();
     bounds.set_y(bounds.y() + bounds.height() - new_height);
     bounds.set_height(new_height);
-    keyboard->SetBounds(bounds);
+    // Keyboard bounds should only be reset when it actually changes. Otherwise
+    // it interrupts the initial animation of showing the keyboard. Described in
+    // crbug.com/356753.
+    if (bounds != keyboard->bounds())
+      keyboard->SetBounds(bounds);
   }
 
   // Overridden from content::WebContentsDelegate:
   virtual void RequestMediaAccessPermission(content::WebContents* web_contents,
       const content::MediaStreamRequest& request,
       const content::MediaResponseCallback& callback) OVERRIDE {
     proxy_->RequestAudioInput(web_contents, request, callback);
   }
 
   // Overridden from content::WebContentsObserver:
@@ skipping 125 matching lines @@
   }
 
   shadow_->SetContentBounds(new_bounds);
 }
 
 void KeyboardControllerProxy::OnWindowDestroyed(aura::Window* window) {
   window->RemoveObserver(this);
 }
 
 }  // namespace keyboard