Closed Bug 1234589 Opened 8 years ago Closed 8 years ago

Update the documentation for shutdownDuration

Categories

(Toolkit :: Telemetry, defect, P2)

defect
Points:
1

Tracking

()

RESOLVED FIXED
mozilla48
Tracking Status
firefox46 --- affected
firefox48 --- fixed

People

(Reporter: Dexter, Assigned: Dexter)

References

Details

(Whiteboard: [measurement:client])

Attachments

(1 file)

Bug 1232285 landed and now we have a clearer definition of |shutdownDuration| in the main ping. We need to update [0] accordingly.

[0] - https://dxr.mozilla.org/mozilla-central/rev/388bdc46ba51ee31da8b8abe977e0ca38d117434/toolkit/components/telemetry/docs/main-ping.rst#147
Blocks: 1216191
Priority: -- → P3
Whiteboard: [measurement:client]
Points: --- → 1
Priority: P3 → P2
Attached patch bug1234589.patchSplinter Review
I've changed the description for shutdownDurations, this is always milliseconds [1].

[1] - https://dxr.mozilla.org/mozilla-central/rev/4037eb98974db1b1e0b5012c8a7f3a36428eaa11/toolkit/components/telemetry/Telemetry.cpp#1681,3847
Assignee: nobody → alessio.placitelli
Status: NEW → ASSIGNED
Attachment #8733300 - Flags: review?(gfritzsche)
Attachment #8733300 - Flags: review?(gfritzsche) → review+
This only updates the documentation, no try push needed.
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/f1c04c3220a4
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in before you can comment on or make changes to this bug.