Add the app.windows.getBounds method

chrome/renderer/resources/extensions/app_window_custom_bindings.js

 // Copyright (c) 2012 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.
 
 // Custom bindings for the app_window API.
 
 var chromeHidden = requireNative('chrome_hidden').GetChromeHidden();
 var sendRequest = require('sendRequest').sendRequest;
 var appWindowNatives = requireNative('app_window');
 var forEach = require('utils').forEach;
@@ skipping 65 matching lines @@
       AppWindow.prototype[fn] =
           chromeHidden.internalAPIs.app.currentWindowInternal[fn];
     });
     AppWindow.prototype.moveTo = window.moveTo.bind(window);
     AppWindow.prototype.resizeTo = window.resizeTo.bind(window);
     AppWindow.prototype.contentWindow = window;
     AppWindow.prototype.onClosed = new chrome.Event;
     AppWindow.prototype.close = function() {
       this.contentWindow.close();
     };
+    AppWindow.prototype.getBounds = function() {
+      var bounds = chromeHidden.appWindowData.bounds;
+      return { left: bounds.left, top: bounds.top,
+               width: bounds.width, height:bounds.height };

[jeremya, 2012/10/31 00:32:46]

nit: missing space

[asargent_no_longer_on_chrome, 2012/10/31 05:37:35]

Done.

+    };
 
     Object.defineProperty(AppWindow.prototype, 'id', {get: function() {
       return chromeHidden.appWindowData.id;
     }});
 
     chromeHidden.appWindowData = {
-      id: params.id || ''
+      id: params.id || '',
+      bounds: { left: 0, top: 0, width: 0, height: 0 }
     };
     chromeHidden.currentAppWindow = new AppWindow;
   });
 });
+
+chromeHidden.updateAppWindowBounds = function(info) {
+  var data = chromeHidden.appWindowData;
+  if (!data)
+    return;
+  data.bounds.left = info.left;
+  data.bounds.top = info.top;
+  data.bounds.width = info.width;
+  data.bounds.height = info.height;
+};