Bug 1274727 - Use toString method for search engine identifiers. r=bnicholson
These values are what are actually used UI telemetry (perhaps because the enums were added after the strings?). They're also a little less obnoxious than the enum names.
MozReview-Commit-ID: K5i2Hr4DR4J
--- a/mobile/android/base/java/org/mozilla/gecko/BrowserApp.java
+++ b/mobile/android/base/java/org/mozilla/gecko/BrowserApp.java
@@ -2435,17 +2435,17 @@ public class BrowserApp extends GeckoApp
*
* @param where where the search was started from
*/
private static void recordSearch(@NonNull final SharedPreferences prefs, @NonNull final String engineIdentifier,
@NonNull final TelemetryContract.Method where) {
// We could include the engine identifier as an extra but we'll
// just capture that with core ping telemetry (bug 1253319).
Telemetry.sendUIEvent(TelemetryContract.Event.SEARCH, where);
- SearchCountMeasurements.incrementSearch(prefs, engineIdentifier, where.name());
+ SearchCountMeasurements.incrementSearch(prefs, engineIdentifier, where.toString());
}
/**
* Store search query in SearchHistoryProvider.
*
* @param query
* a search query to store. We won't store empty queries.
*/