Closed
Bug 1384511
Opened 7 years ago
Closed 7 years ago
about:telemetry histograms section shows wrong histograms
Categories
(Toolkit :: Telemetry, defect, P1)
Toolkit
Telemetry
Tracking
()
RESOLVED
FIXED
mozilla56
Tracking | Status | |
---|---|---|
firefox56 | --- | fixed |
People
(Reporter: gfritzsche, Assigned: flyingrub)
References
Details
Attachments
(1 file)
When going to about:telemetry, then selecting a section for which no histograms are available, it will break (or after bug 1302681 fall back to the parent ones). https://dxr.mozilla.org/mozilla-central/rev/7d2e89fb92331d7e4296391213c1e63db628e046/toolkit/content/aboutTelemetry.js#2085 This needs some cleanup: - the section "hasData" when *any* of `payload.parent.histograms` or `payload.processes.<P>.histograms` exist and are non-empty - this should not fall back to the parent histograms - for empty processes this should either 1) disable the process entry or 2) just show empty tables
Reporter | ||
Comment 1•7 years ago
|
||
The same goes for the keyed histograms section.
Comment hidden (mozreview-request) |
Comment 3•7 years ago
|
||
mozreview-review |
Comment on attachment 8891396 [details] Bug 1384511 - Display the correct histograms in about:telemetry https://reviewboard.mozilla.org/r/162564/#review167838 Looks like it should work.
Attachment #8891396 -
Flags: review?(chutten) → review+
Keywords: checkin-needed
Reporter | ||
Comment 4•7 years ago
|
||
(In reply to flyingrub from comment #2) > Created attachment 8891396 [details] > Bug 1384511 - Display the correct histograms in about:telemetry > > Fix "has-data" for all section that uses the process select. > Display no histograms when the current process hasn't any data and not > the parent ones. Does this mean i should drop the "part 2" patch in bug 1302681?
Flags: needinfo?(flyinggrub)
Yup, I think it will probably only causes merge conflicts :S
Flags: needinfo?(flyinggrub)
Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/autoland/rev/ceb431d84aab Display the correct histograms in about:telemetry r=chutten
Keywords: checkin-needed
Comment 7•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/ceb431d84aab
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla56
You need to log in
before you can comment on or make changes to this bug.
Description
•