diff --git a/browser/base/content/browser.js b/browser/base/content/browser.js index 92a2046b80f9..95491e3dc872 100755 --- a/browser/base/content/browser.js +++ b/browser/base/content/browser.js @@ -3964,7 +3964,8 @@ const BrowserSearch = { return engine.identifier; } - if (!engine || (engine.name === undefined)) + if (!engine || (engine.name === undefined) || + !Services.prefs.getBoolPref("toolkit.telemetry.enabled")) return "other"; return "other-" + engine.name; diff --git a/browser/modules/BrowserUsageTelemetry.jsm b/browser/modules/BrowserUsageTelemetry.jsm index 3b5872f4ad53..b3497b7931df 100644 --- a/browser/modules/BrowserUsageTelemetry.jsm +++ b/browser/modules/BrowserUsageTelemetry.jsm @@ -112,7 +112,12 @@ function getSearchEngineId(engine) { if (engine.identifier) { return engine.identifier; } - if (engine.name) { + // Due to bug 1222070, we can't directly check Services.telemetry.canRecordExtended + // here. + const extendedTelemetry = Services.prefs.getBoolPref("toolkit.telemetry.enabled"); + if (engine.name && extendedTelemetry) { + // If it's a custom search engine only report the engine name + // if extended Telemetry is enabled. return "other-" + engine.name; } }