Closed Bug 1234589 Opened 10 years ago Closed 10 years ago

Update the documentation for shutdownDuration

Categories

(Toolkit :: Telemetry, defect, P2)

Product:
Toolkit
Component:
Telemetry
Type:
defect
Points:
1

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla48
Tracking Flags:
Tracking Status
firefox46 --- affected
firefox48 --- fixed

People

(Reporter: Dexter, Assigned: Dexter)

References

Details

(Whiteboard: [measurement:client])

Attachments

(1 file)

bug1234589.patch
1.18 KB, patch
gfritzsche
: review+
Details | Diff | Splinter Review
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.patch — — Splinter 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: 10 years ago
status-firefox48: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: