Test failure fix. No bug, but change talked over with mak on irc.
authorShawn Wilsher <me@shawnwilsher.com>
Mon, 18 Oct 2010 08:53:53 -0700
changeset 59308 05ec200ea14bc797bc787dfa61aaa193274971b0
parent 59307 c25ad89fe707bb1cb00888b7952ee3c869ab2347
child 59309 ccef12b1fe7eec108b2c0d744e84a5f460969136
push id1
push usershaver@mozilla.com
push dateTue, 04 Jan 2011 17:58:04 +0000
milestone2.0b8pre
Test failure fix. No bug, but change talked over with mak on irc.
toolkit/components/places/tests/unit/test_history_removeAllPages.js
--- a/toolkit/components/places/tests/unit/test_history_removeAllPages.js
+++ b/toolkit/components/places/tests/unit/test_history_removeAllPages.js
@@ -163,20 +163,16 @@ let historyObserver = {
 
   QueryInterface: XPCOMUtils.generateQI([
     Ci.nsINavHistoryObserver,
   ]),
 }
 PlacesUtils.history.addObserver(historyObserver, false);
 
 function run_test() {
-  // Since we are checking frecency, we have to disable some stuff that could
-  // change values during test run.  On idle-daily frecencies are updated.
-  Services.obs.removeObserver(PlacesUtils.history, "idle-daily");
-
   // places-init-complete is notified after run_test, and it will
   // run a first frecency fix through async statements.
   // To avoid random failures we have to run after all of this.
   Services.obs.addObserver(function(aSubject, aTopic, aData) {
     Services.obs.removeObserver(arguments.callee, aTopic, false);
     do_execute_soon(continue_test);
   }, PlacesUtils.TOPIC_INIT_COMPLETE, false);