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);
+    if (bounds != keyboard->bounds())

[bshe, 2014/09/30 21:57:16]

nit: It would be nice to add a comment on why this check is necessary. And just
out of curiosity, do we call move contents multiple times with one
window.resizeTo or window.resizeTo will trigger this function being called
multiple times?

+      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