Disable Places Sync on some frequent random failing test, to workaround bug 507790 and reduce orangeness. a=tests-only
authorMarco Bonardo <mbonardo@mozilla.com>
Thu, 26 Nov 2009 15:29:03 +0100
changeset 35248 12b704972248c52bdf11560d3704549d077f7ded
parent 35247 8ae42c1b8d83dae43bcf78dc53b12577efc9f3c4
child 35249 a293d150ebbd9dca19b144d23510661225031951
push id10523
push usermak77@bonardo.net
push dateThu, 26 Nov 2009 14:29:37 +0000
treeherdermozilla-central@12b704972248 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerstests-only
bugs507790
milestone1.9.3a1pre
Disable Places Sync on some frequent random failing test, to workaround bug 507790 and reduce orangeness. a=tests-only
toolkit/components/places/tests/queries/head_queries.js
toolkit/components/places/tests/unit/head_bookmarks.js
--- a/toolkit/components/places/tests/queries/head_queries.js
+++ b/toolkit/components/places/tests/queries/head_queries.js
@@ -562,15 +562,17 @@ function flush_main_thread_events()
   while (tm.mainThread.hasPendingEvents())
     tm.mainThread.processNextEvent(false);
 }
 
 // These tests are known to randomly fail due to bug 507790 when database
 // flushes are active, so we turn off syncing for them.
 let randomFailingSyncTests = [
   "test_results-as-visit.js",
+  "test_sorting.js",
+  "test_redirectsMode.js",
 ];
 let currentTestFilename = do_get_file(_TEST_FILE[0], true).leafName;
 if (randomFailingSyncTests.indexOf(currentTestFilename) != -1) {
   print("Test " + currentTestFilename + " is known random due to bug 507790, disabling PlacesDBFlush component.");
   let sync = Cc["@mozilla.org/places/sync;1"].getService(Ci.nsIObserver);
   sync.observe(null, "places-debug-stop-sync", null);
 }
--- a/toolkit/components/places/tests/unit/head_bookmarks.js
+++ b/toolkit/components/places/tests/unit/head_bookmarks.js
@@ -300,15 +300,17 @@ function flush_main_thread_events()
 // These tests are known to randomly fail due to bug 507790 when database
 // flushes are active, so we turn off syncing for them.
 let randomFailingSyncTests = [
   "test_annotations.js",
   "test_multi_word_tags.js",
   "test_removeVisitsByTimeframe.js",
   "test_tagging.js",
   "test_utils_getURLsForContainerNode.js",
+  "test_exclude_livemarks.js",
+  "test_402799.js",
 ];
 let currentTestFilename = do_get_file(_TEST_FILE[0], true).leafName;
 if (randomFailingSyncTests.indexOf(currentTestFilename) != -1) {
   print("Test " + currentTestFilename + " is known random due to bug 507790, disabling PlacesDBFlush component.");
   let sync = Cc["@mozilla.org/places/sync;1"].getService(Ci.nsIObserver);
   sync.observe(null, "places-debug-stop-sync", null);
 }