Closed
Bug 982818
Opened 11 years ago
Closed 11 years ago
about:telemetry displays startupInterrupted and debuggerAttached incorrectly
Categories
(Toolkit :: Telemetry, defect)
Tracking
()
RESOLVED
FIXED
mozilla30
People
(Reporter: Irving, Assigned: Irving)
References
Details
Attachments
(1 file)
|
(deleted),
patch
|
vladan
:
review+
|
Details | Diff | Splinter Review |
The telemetry field formatter added as part of bug 846921 causes the Simple Measurements: startupInterrupted and debuggerAttached fields to display as {} rather than 0/1, because these values are passed as JS Number objects rather than unboxed numbers. Though we don't currently have any, RenderObject() will handle objects created with 'new String()' badly too.
|
Assignee | |
Comment 1•11 years ago
|
||
Fixed both ways - RenderObject() does the right thing for boxed types, and TelemetryPing.jsm uses Number to convert booleans to unboxed numbers rather than creating new boxed Number objects.
Attachment #8390130 -
Flags: review?(vdjeric)
|
||
Updated•11 years ago
|
Attachment #8390130 -
Flags: review?(vdjeric) → review+
|
|
Assignee | |
Updated•11 years ago
|
Keywords: checkin-needed
|
||
Comment 2•11 years ago
|
||
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → mozilla30
You need to log in
before you can comment on or make changes to this bug.
Description
•