Adjust virtual keyboard size base on input method candidates.

chrome/browser/resources/keyboard/main.js

Index: chrome/browser/resources/keyboard/main.js
diff --git a/chrome/browser/resources/keyboard/main.js b/chrome/browser/resources/keyboard/main.js
index cf3f97a93505f5f8e95c1037f793d0255ff41f03..b440935f313e8517502df8faa6eb5a8c33b1a00a 100644
--- a/chrome/browser/resources/keyboard/main.js
+++ b/chrome/browser/resources/keyboard/main.js
@@ -578,7 +578,7 @@ var allRows = [];  // Populated during start()
  */
 function getRowHeight() {
   var x = window.innerWidth;
-  var y = window.innerHeight - (imeui ? IME_HEIGHT - 2 : 0);
+  var y = window.innerHeight - ((imeui && imeui.visible) ? IME_HEIGHT : 0);
   return (x > kKeyboardAspect * y) ?
       (height = Math.floor(y / 4)) :
       (height = Math.floor(x / (kKeyboardAspect * 4)));
@@ -647,6 +647,8 @@ function sendKeyFunction(key) {
   }
 }
 
+var oldHeight = 0;
+var oldX = 0;
 /**
  * Resize the keyboard according to the new window size.
  * @return {void}
@@ -655,16 +657,23 @@ window.onresize = function() {
   var height = getRowHeight();
   var newX = document.documentElement.clientWidth;
 
-  // All rows should have the same aspect, so just use the first one
-  var totalWidth = Math.floor(height * allRows[0].aspect);
-  var leftPadding = Math.floor((newX - totalWidth) / 2);
-  document.getElementById('b').style.paddingLeft = leftPadding + 'px';
+  if (newX != oldX || height != oldHeight) {

[bryeung, 2011/06/07 22:45:03]

Sorry: I'm still not understanding.

Are you saying that window.onresize can be called even if the height has not
changed?  Or is that the height could have changed, but it might be offset by
the IME height, so no resize is necessary?  i.e. the extra window height is
exactly taken up by the candidates, so we don't need to resize the keys?

[Peng, 2011/06/08 14:59:57]

Yeah. Sometime, the window.onresize is caused by showing or hiding candidates.
In those case, the height of keys should be unchanged. So they do not need
resize.

+    // All rows should have the same aspect, so just use the first one
+    var totalWidth = Math.floor(height * allRows[0].aspect);
+    var leftPadding = Math.floor((newX - totalWidth) / 2);
+    document.getElementById('b').style.paddingLeft = leftPadding + 'px';
+  }
 
-  for (var i = 0; i < allRows.length; ++i) {
-    allRows[i].resize(height);
+  if (height != oldHeight) {
+    for (var i = 0; i < allRows.length; ++i) {
+      allRows[i].resize(height);
+    }
   }
 
   updateIme();
+
+  oldHeight = height;
+  oldX = newX;
 }
 
 /**
@@ -680,10 +689,12 @@ window.onload = function() {
     allRows.push(new Row(i, KEYS[i]));
   }
 
+  height = getRowHeight();
   for (var i = 0; i < allRows.length; ++i) {
-    body.appendChild(allRows[i].makeDOM(getRowHeight()));
+    body.appendChild(allRows[i].makeDOM(height));
     allRows[i].showMode(KEY_MODE);
   }
+  oldHeight = height;
 
   window.onresize();
 }