Bug 534647 - mochitest-browser-chrome: perma/random "browser_ApplicationPrefs.js | Timed out" after bug 152526 landing; (Fv1-FF) Improve a few log messages.
authorSerge Gautherie <sgautherie.bz@free.fr>
Sun, 24 Jan 2010 14:58:31 +0100
changeset 37453 56a74ab99f43797428cac3dd7f6261d8438e210b
parent 37452 97c7954d972dcad80196352e3f49f745597508b6
child 37454 fb61277403e38a9b2d6dee7a9f9bef7e3d7ad1f0
push id11311
push usersgautherie.bz@free.fr
push dateSun, 24 Jan 2010 13:59:03 +0000
treeherdermozilla-central@fb61277403e3 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs534647, 152526
milestone1.9.3a1pre
Bug 534647 - mochitest-browser-chrome: perma/random "browser_ApplicationPrefs.js | Timed out" after bug 152526 landing; (Fv1-FF) Improve a few log messages. r=gavin.sharp.
browser/fuel/test/browser_ApplicationPrefs.js
--- a/browser/fuel/test/browser_ApplicationPrefs.js
+++ b/browser/fuel/test/browser_ApplicationPrefs.js
@@ -135,43 +135,43 @@ function test() {
   ok(!pref.locked, "A single preference should not be locked.");
 
   pref.locked = true;
   ok(pref.locked, "A single preference should be locked.");
 
   try {
     prev.value = "test value";
 
-    ok(false, "A locked preference should not be able to be modified.");
+    ok(false, "A locked preference could be modified.");
   } catch(e){
     ok(true, "A locked preference should not be able to be modified.");
   }
 
   pref.locked = false;
-  ok(!pref.locked, "A single preference should not be locked.");
+  ok(!pref.locked, "A single preference is unlocked.");
 
   // check for change event when setting a value
   waitForExplicitFinish();
   Application.prefs.events.addListener("change", onPrefChange);
   Application.prefs.setValue("fuel.fuel-test", "change event");
 }
 
 function onPrefChange(evt) {
-  is(evt.data, testdata.dummy, "Check 'Application.prefs.set' fired a change event");
+  is(evt.data, testdata.dummy, "Check 'Application.prefs.setValue' fired a change event");
   Application.prefs.events.removeListener("change", onPrefChange);
 
   // We are removing the old listener after adding the new listener so we can test that
   // removing a listener does not remove all listeners
   Application.prefs.get("fuel.fuel-test").events.addListener("change", onPrefChangeDummy);
   Application.prefs.get("fuel.fuel-test").events.addListener("change", onPrefChange2);
   Application.prefs.get("fuel.fuel-test").events.removeListener("change", onPrefChangeDummy);
 
   Application.prefs.setValue("fuel.fuel-test", "change event2");
 }
 
 function onPrefChange2(evt) {
-  is(evt.data, testdata.dummy, "Check 'Application.prefs.set' fired a change event for a single preference");
+  is(evt.data, testdata.dummy, "Check 'Application.prefs.setValue' fired a change event for a single preference");
   Application.prefs.events.removeListener("change", onPrefChange2);
 
   finish();
 }
 
 function onPrefChangeDummy(evt) { }