b=797975, disable browser_tabview_bug587503 test 4 because it's fragile
authorVladimir Vukicevic <vladimir@pobox.com>
Thu, 04 Oct 2012 19:58:09 -0400
changeset 109398 2bdb3c531c3ef4062edbb330c6a3dccbd416bcbc
parent 109397 4e4f65c5f43b893ea1e097760496d0e609758985
child 109399 739aff49b8bb7ab8971c874c8212f5d62a8bcd45
push id23630
push useremorley@mozilla.com
push dateSat, 06 Oct 2012 19:35:27 +0000
treeherdermozilla-central@9f677c2bb33d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs797975, 587503
milestone18.0a1
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
b=797975, disable browser_tabview_bug587503 test 4 because it's fragile
browser/components/tabview/test/browser_tabview_bug587503.js
--- a/browser/components/tabview/test/browser_tabview_bug587503.js
+++ b/browser/components/tabview/test/browser_tabview_bug587503.js
@@ -44,23 +44,28 @@ function test() {
     is(getTabNumbers(aGroup), "0,6,2,3,4,5,1", "Validate tab positions in test 2.");
 
     // Test 3: Move the fifth tab outside the group.
     tab = aGroup.getChild(4);
     moveTabOutOfGroup(tab, aGroup, aCW);
     is(getTabNumbers(aGroup), "0,6,2,3,5,1", "Validate tab positions in test 3.");
     is(aCW.GroupItems.groupItems.length, 3, "Validate group count in test 3.");
 
-    // Test 4: Move the fifth tab back into the group, on the second row.
-    waitForTransition(tab, function() {
-      moveTabInGroup(tab, 4, aGroup, aCW);
-      is(getTabNumbers(aGroup), "0,6,2,3,4,5,1", "Validate tab positions in test 4.");
-      is(aCW.GroupItems.groupItems.length, 2, "Validate group count in test 4.");
+    // This test is disabled because it is fragile -- see bug 797975
+    if (false) {
+      // Test 4: Move the fifth tab back into the group, on the second row.
+      waitForTransition(tab, function() {
+	moveTabInGroup(tab, 4, aGroup, aCW);
+	is(getTabNumbers(aGroup), "0,6,2,3,4,5,1", "Validate tab positions in test 4.");
+	is(aCW.GroupItems.groupItems.length, 2, "Validate group count in test 4.");
+	closeGroupItem(aGroup, function() { hideTabView(finish) });
+      });
+    } else {
       closeGroupItem(aGroup, function() { hideTabView(finish) });
-    });
+    }
   }
 
   function createGroup(win) {
     registerCleanupFunction(function() win.close());
     let cw = win.TabView.getContentWindow();
     let group = createGroupItemWithTabs(win, 400, 430, 100, [
       "about:blank#0", "about:blank#1", "about:blank#2", "about:blank#3",
       "about:blank#4", "about:blank#5", "about:blank#6"