Bug 1102416 followup #2: fix tests, r+a=gavin
authorGavin Sharp <gavin@gavinsharp.com>
Tue, 25 Nov 2014 22:08:32 -0800
changeset 227449 62287d20bf35664cbe47d6fc8b4db055e526dc0c
parent 227448 f2f39afa775b4b45ecba4f6578214d764b040ffa
child 227452 1975d9ea9dc16ad2f5e3d3c69871d83c79ce583e
push id596
push usergsharp@mozilla.com
push dateWed, 26 Nov 2014 06:08:42 +0000
treeherdermozilla-release@62287d20bf35 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1102416
milestone34.0
Bug 1102416 followup #2: fix tests, r+a=gavin
browser/base/content/test/general/browser_contextSearchTabPosition.js
--- a/browser/base/content/test/general/browser_contextSearchTabPosition.js
+++ b/browser/base/content/test/general/browser_contextSearchTabPosition.js
@@ -48,18 +48,19 @@ function test() {
     ok(provider, "Searches provider is available.");
 
     let m = provider.getMeasurement("counts", 3);
     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 to be changed if Google isn't the default search engine.
-      let field = "google.contextmenu";
+      // Will need to be changed if Yahoo isn't the default search engine.
+      let defaultProviderID = "yahoo";
+      let field = defaultProviderID + ".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();
     });