Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(190)

Unified Diff: chrome/test/data/extensions/platform_apps/windows_api/test.js

Issue 10436015: Remove chrome.windows.* support for platform apps. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/test/data/extensions/platform_apps/windows_api/test.html ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/test/data/extensions/platform_apps/windows_api/test.js
diff --git a/chrome/test/data/extensions/platform_apps/windows_api/test.js b/chrome/test/data/extensions/platform_apps/windows_api/test.js
index 2e71c9bf21d0d0f67e1e2216fb27a217c528cd9d..09e3dbf6f52eae42f7d58b7e19023a39c467a57d 100644
--- a/chrome/test/data/extensions/platform_apps/windows_api/test.js
+++ b/chrome/test/data/extensions/platform_apps/windows_api/test.js
@@ -5,51 +5,26 @@
var callbackPass = chrome.test.callbackPass;
chrome.experimental.app.onLaunched.addListener(function() {
- // Only a few chrome.windows.* API methods are tested, it is assumed that the
- // full API is tested elsewhere.
chrome.test.runTests([
function testCreateWindow() {
- var getShellWindowCount = function(callback) {
- chrome.windows.getAll(function(windows) {
- callback(windows.filter(
- function(window) { return window.type == 'shell' }).length);
- });
- };
-
- getShellWindowCount(callbackPass(function(shellWindowCount) {
- chrome.test.assertEq(0, shellWindowCount);
- chrome.windows.create(
- {type: 'shell', width: 128, height: 128},
- callbackPass(function() {
- getShellWindowCount(callbackPass(function(shellWindowCount) {
- chrome.test.assertEq(1, shellWindowCount);
- }));
- }));
- }));
- },
-
- function testUpdateWindowWidth() {
- chrome.windows.create(
- {type: 'shell', width: 128, height: 128},
- callbackPass(function(window) {
- chrome.windows.update(
- window.id,
- // 256 is a width that will snap to the Aura grid, thus this
- // should be the actual width on Aura (when read back).
- {width: 256, height: 128},
- callbackPass(function() {
- // The timeout is rather lame, but reading back the width from
- // the window manager (on Linux) immediately still reports the
- // old value.
- setTimeout(callbackPass(function() {
- chrome.windows.getCurrent(callbackPass(function(window) {
- chrome.test.assertEq(256, window.width);
- }));
- }), 1000);
- }));
- }));
- },
-
+ chrome.appWindow.create('test.html', {}, callbackPass(function (win) {
+ chrome.test.assertTrue(typeof win.window === 'object');
+ }))
+ },
+ function testUpdateWindowWidth() {
+ chrome.appWindow.create('test.html',
+ {width:512, height:384, frame:'custom'},
+ callbackPass(function(win) {
+ // TODO(jeremya): assert that width and height specified in options
+ // above specify the innerWidth, not the outerWidth.
+ //chrome.test.assertEq(512, win.outerWidth);
+ //chrome.test.assertEq(384, win.outerHeight);
+ var oldWidth = win.outerWidth, oldHeight = win.outerHeight;
+ win.resizeBy(-256, 0);
+ chrome.test.assertEq(oldWidth - 256, win.outerWidth);
+ chrome.test.assertEq(oldHeight, win.outerHeight);
+ }));
+ },
]);
});
« no previous file with comments | « chrome/test/data/extensions/platform_apps/windows_api/test.html ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698