From 2407f02151c30f7a54194df5d49eb99e46b9af07 Mon Sep 17 00:00:00 2001 From: Andrew Halberstadt Date: Thu, 7 Sep 2017 11:38:51 -0400 Subject: [PATCH] Bug 1397229 - Remove unused reflow from browser_windowopen_reflows.js on Windows, r=mconley This test was previously skipped on Windows 8. This bug is migrating the test over to Windows 10 where we discovered the unused reflow. MozReview-Commit-ID: FqmgeKc1b9o --HG-- extra : rebase_source : 4c15104b9d63c44dc5c397cb34cf9320797a48a4 --- browser/base/content/test/performance/browser.ini | 2 +- .../test/performance/browser_windowopen_reflows.js | 12 ------------ 2 files changed, 1 insertion(+), 13 deletions(-) diff --git a/browser/base/content/test/performance/browser.ini b/browser/base/content/test/performance/browser.ini index b2546281e033..347fa9af5862 100644 --- a/browser/base/content/test/performance/browser.ini +++ b/browser/base/content/test/performance/browser.ini @@ -21,4 +21,4 @@ skip-if = (os == 'linux') || (os == 'win' && debug) # Disabled on Linux and Wind skip-if = (os == 'linux') || (os == 'mac' && !debug) # Disabled on Linux and OS X opt due to frequent failures. Bug 1385932 and Bug 1384582 [browser_windowclose_reflows.js] [browser_windowopen_reflows.js] -skip-if = os == 'linux' # Disabled due to frequent failures. Bug 1380465. +skip-if = os == 'linux' # Disabled due to frequent failures. bug 1380465 diff --git a/browser/base/content/test/performance/browser_windowopen_reflows.js b/browser/base/content/test/performance/browser_windowopen_reflows.js index dc9a0177d85f..29c2cabd8417 100644 --- a/browser/base/content/test/performance/browser_windowopen_reflows.js +++ b/browser/base/content/test/performance/browser_windowopen_reflows.js @@ -68,12 +68,6 @@ if (Services.appinfo.OS == "WINNT") { "handleEvent@chrome://browser/content/browser.js", ], }, - - { - stack: [ - "handleEvent@chrome://browser/content/tabbrowser.xml", - ], - } ); } @@ -106,12 +100,6 @@ if (Services.appinfo.OS == "WINNT" || Services.appinfo.OS == "Darwin") { * uninterruptible reflows when opening new windows. */ add_task(async function() { - const IS_WIN8 = (navigator.userAgent.indexOf("Windows NT 6.2") != -1); - if (IS_WIN8) { - ok(true, "Skipping this test because of perma-failures on Windows 8 x64 (bug 1381521)"); - return; - } - // Flushing all caches helps to ensure that we get consistent // behaviour when opening a new window, even if windows have been // opened in previous tests.