Closed Bug 1319788 Opened 3 years ago Closed 3 years ago
Telemetry for number of imported items is broken
(In reply to :Gijs Kruitbosch from bug 1279501 comment #20) > I am an idiot. > > > let histogram = Services.telemetry.getKeyedHistogram(histogramId); > > should be getKeyedHistogramById > > Also bug 1311100's fixes can't come soon enough. > > I'll file a dep to get this fixed and uplifted ASAP.
Comment on attachment 8813683 [details] Bug 1319788 - fix typo in getting a telemetry histogram, https://reviewboard.mozilla.org/r/95092/#review95248
Attachment #8813683 - Flags: review?(mak77) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/630e0a8c3241 fix typo in getting a telemetry histogram, r=mak
Comment on attachment 8813683 [details] Bug 1319788 - fix typo in getting a telemetry histogram, Approval Request Comment [Feature/regressing bug #]: bug 1279501 [User impact if declined]: no telemetry, no decisions, bad for users [Describe test coverage new/current, TreeHerder]: no, because telemetry. We'll know it works when telemetry comes in (also, I tested it locally, for whatever that means). [Risks and why]: none - it was broken and not actually breaking anything (besides the telemetry) because of the try catch. This can only make things better, not worse. [String/UUID change made/needed]: nope
Comment on attachment 8813683 [details] Bug 1319788 - fix typo in getting a telemetry histogram, Fix a typo error in telemetry. Beta51+ and Aurora52+. Should be in 51 beta 3.
You need to log in before you can comment on or make changes to this bug.