Backed out changeset 1a7d17e9e15d (bug 1276738)
authorCarsten "Tomcat" Book <cbook@mozilla.com>
Thu, 02 Jun 2016 08:12:35 +0200
changeset 341066 4272db827c1baaa96d1cf7b44680fa56fcd55f3c
parent 341065 59583eadba52a1af7670b2d9908e8cdf991186c3
child 341067 105fffd482d1e298ebf6b0dfa27a74021814cc13
push id1183
push userraliiev@mozilla.com
push dateMon, 05 Sep 2016 20:01:49 +0000
treeherdermozilla-release@3148731bed45 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1276738
milestone49.0a1
backs out1a7d17e9e15d84f22d56a056cb48c28b3ba23c7b
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Backed out changeset 1a7d17e9e15d (bug 1276738)
embedding/components/windowwatcher/test/browser.ini
embedding/components/windowwatcher/test/browser_new_sized_window.js
--- a/embedding/components/windowwatcher/test/browser.ini
+++ b/embedding/components/windowwatcher/test/browser.ini
@@ -1,7 +1,5 @@
 [DEFAULT]
 tags = openwindow
 
 [browser_new_remote_window_flags.js]
 run-if = e10s
-[browser_new_sized_window.js]
-skip-if = os == 'win' # Bug 1276802 - Opening windows from content on Windows might not get the size right
deleted file mode 100644
--- a/embedding/components/windowwatcher/test/browser_new_sized_window.js
+++ /dev/null
@@ -1,67 +0,0 @@
-"use strict";
-
-/**
- * Tests that content can open windows at requested dimensions
- * of height and width.
- */
-
-/**
- * This utility function does most of the actual testing. We
- * construct a feature string suitable for the passed in width
- * and height, and then run that script in content to open the
- * new window. When the new window comes up, this function tests
- * to ensure that the content area of the initial browser is the
- * requested dimensions. Finally, we also ensure that we're not
- * persisting the position, size or sizemode of the new browser
- * window.
- */
-function test_dimensions({ width, height}) {
-  let features = [];
-  if (width) {
-    features.push(`width=${width}`);
-  }
-  if (height) {
-    features.push(`height=${height}`);
-  }
-  const FEATURE_STR = features.join(",");
-  const SCRIPT_PAGE = `data:text/html,<script>window.open("about:blank", "_blank", "${FEATURE_STR}");</script>`;
-
-  let newWinPromise = BrowserTestUtils.waitForNewWindow();
-
-  return BrowserTestUtils.withNewTab({
-    gBrowser,
-    url: SCRIPT_PAGE,
-  }, function*(browser) {
-    let win = yield newWinPromise;
-    let rect = win.gBrowser.selectedBrowser.getBoundingClientRect();
-
-    if (width) {
-      Assert.equal(rect.width, width, "Should have the requested width");
-    }
-
-    if (height) {
-      Assert.equal(rect.height, height, "Should have the requested height");
-    }
-
-    let treeOwner = win.QueryInterface(Ci.nsIInterfaceRequestor)
-                       .getInterface(Ci.nsIDocShell)
-                       .QueryInterface(Ci.nsIDocShellTreeItem)
-                       .treeOwner;
-    let persistPosition = {};
-    let persistSize = {};
-    let persistSizeMode = {};
-    treeOwner.getPersistence(persistPosition, persistSize, persistSizeMode);
-
-    Assert.ok(!persistPosition.value, "Should not persist position");
-    Assert.ok(!persistSize.value, "Should not persist size");
-    Assert.ok(!persistSizeMode.value, "Should not persist size mode");
-
-    yield BrowserTestUtils.closeWindow(win);
-  });
-}
-
-add_task(function* test_new_sized_window() {
-  yield test_dimensions({ width: 100 });
-  yield test_dimensions({ height: 150 });
-  yield test_dimensions({ width: 300, height: 200 });
-});