Merge pull request #33009 from tamarahills/bugfix/1221867-fix-custom-…

…hist-with-multiple-underscores Bug 1221867 - Use a regexp to globally replace underscores in histogr…
mozilla-b2g · Nov 5, 2015 · 65973b6 · 65973b6
2 parents f39a7a8 + 4fdb6cc
commit 65973b6
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/shared/js/advanced_telemetry_helper.js b/shared/js/advanced_telemetry_helper.js
@@ -130,9 +130,10 @@
       this.metricType = type;
       if (typeof name !== 'undefined') {
         var varCount = 2;
+        var re = /_/g;
         var message = APP_TELEMETRY_LOG_PREFIX;
         // Disallow underscores so telemetry system can parse it.
-        this.metricName = name.replace('_', '-');
+        this.metricName = name.replace(re, '-');
         message += '|' + this.metricName;
         message += '|' + type + '|';
         if (typeof min !== 'undefined') { varCount++; }