Bug 841554 - Part 3: follow-up fix for another test failure. r=trivial, a=bustage
authorRichard Newman <rnewman@mozilla.com>
Fri, 29 Mar 2013 14:40:54 -0700
changeset 128729 fe2e51f95fdfa53a6c43c0136eed5f7a4e4c738a
parent 128728 2174a373f5d6e8a344f48ae594bdc95c8e8c2897
child 128730 974d7b407fd21af55fd250e4ddcc1aa5f8d9e418
push id3571
push userrnewman@mozilla.com
push dateFri, 29 Mar 2013 21:41:02 +0000
treeherdermozilla-aurora@fe2e51f95fdf [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerstrivial, bustage
bugs841554
milestone21.0a2
Bug 841554 - Part 3: follow-up fix for another test failure. r=trivial, a=bustage
browser/base/content/test/browser_contextSearchTabPosition.js
browser/base/content/test/browser_urlbar_search_healthreport.js
--- a/browser/base/content/test/browser_contextSearchTabPosition.js
+++ b/browser/base/content/test/browser_contextSearchTabPosition.js
@@ -42,23 +42,23 @@ function test() {
                                    .healthReporter;
 
   // reporter should always be available in automation.
   ok(reporter, "Health Reporter available.");
   reporter.onInit().then(function onInit() {
     let provider = reporter.getProvider("org.mozilla.searches");
     ok(provider, "Searches provider is available.");
 
-    let m = provider.getMeasurement("counts", 1);
+    let m = provider.getMeasurement("counts", 2);
     m.getValues().then(function onValues(data) {
       let now = new Date();
       ok(data.days.hasDay(now), "Have data for today.");
       let day = data.days.getDay(now);
 
-      // Will need changed if Google isn't the default search engine.
+      // Will need to be changed if Google isn't the default search engine.
       let field = "google.contextmenu";
       ok(day.has(field), "Have search recorded for context menu.");
 
       // If any other mochitests perform a context menu search, this will fail.
       // The solution will be to look up count at test start and ensure it is
       // incremented by two.
       is(day.get(field), 2, "2 searches recorded in FHR.");
       finish();
--- a/browser/base/content/test/browser_urlbar_search_healthreport.js
+++ b/browser/base/content/test/browser_urlbar_search_healthreport.js
@@ -18,23 +18,23 @@ function test() {
   let reporter = Cc["@mozilla.org/datareporting/service;1"]
                    .getService()
                    .wrappedJSObject
                    .healthReporter;
   ok(reporter, "Health Reporter available.");
   reporter.onInit().then(function onInit() {
     let provider = reporter.getProvider("org.mozilla.searches");
     ok(provider, "Searches provider is available.");
-    let m = provider.getMeasurement("counts", 1);
+    let m = provider.getMeasurement("counts", 2);
 
     m.getValues().then(function onData(data) {
       let now = new Date();
       let oldCount = 0;
 
-      // This will need changed if default search engine is not Google.
+      // This will to be need changed if default search engine is not Google.
       let field = "google.urlbar";
 
       if (data.days.hasDay(now)) {
         let day = data.days.getDay(now);
         if (day.has(field)) {
           oldCount = day.get(field);
         }
       }