Remove redundant SetBounds calls to the keyboard window.

ui/keyboard/keyboard_controller_proxy.cc

Index: ui/keyboard/keyboard_controller_proxy.cc
diff --git a/ui/keyboard/keyboard_controller_proxy.cc b/ui/keyboard/keyboard_controller_proxy.cc
index 1b3e86a27dd58b68164a4c3b8aae61e1129de1f4..a8ff0c159952c6c548b9b42e98938120a28d0a94 100644
--- a/ui/keyboard/keyboard_controller_proxy.cc
+++ b/ui/keyboard/keyboard_controller_proxy.cc
@@ -60,7 +60,8 @@ class KeyboardContentsDelegate : public content::WebContentsDelegate,
     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: