Separate host desktop related functionality into remoting.HostDesktop

remoting/webapp/base/js/window_shape.js

Index: remoting/webapp/base/js/window_shape.js
diff --git a/remoting/webapp/base/js/window_shape.js b/remoting/webapp/base/js/window_shape.js
index dc3ddd109a28433665e381e9406261efc7aa30c0..d4742c5654de635a74362a679d76a8566bd193e9 100644
--- a/remoting/webapp/base/js/window_shape.js
+++ b/remoting/webapp/base/js/window_shape.js
@@ -69,17 +69,6 @@ remoting.WindowShape.prototype.updateClientWindowShape = function() {
   for (var i = 0; i < this.clientUICallbacks_.length; ++i) {
     this.clientUICallbacks_[i].addToRegion(rects);
   }
-  for (var i = 0; i < rects.length; ++i) {
-    var rect = /** @type {ClientRect} */ (rects[i]);
-    var left = Math.floor(rect.left);
-    var right = Math.ceil(rect.left + rect.width);
-    var top = Math.floor(rect.top);
-    var bottom = Math.ceil(rect.top + rect.height);
-    rects[i] = { left: left,
-                 top: top,
-                 width: right - left,
-                 height: bottom - top };
-  }

[Jamie, 2015/02/19 22:44:04]

Apologies for the churn, but now I see the code removed, I think I was wrong
about moving it. It's mostly needed to handle client UI callbacks that return
non-integer values (for example, because they measure DOM elements, which can
have fractional coordinates). Forcing the shape reported by the host desktop to
have integer coordinates is not going to help with that, so I think this part of
the change should be reverted.

[kelvinp, 2015/02/19 23:36:20]

Done.

   chrome.app.window.current().setShape({rects: rects});
 };