Closed Bug 1276198 Opened 8 years ago Closed 8 years ago

Update about:telemetry to show scalars and keyedScalars

Categories

(Toolkit :: Telemetry, defect, P1)

defect
Points:
1

Tracking

()

RESOLVED FIXED
mozilla50
Tracking Status
firefox49 --- affected
firefox50 --- fixed

People

(Reporter: gfritzsche, Assigned: gfritzsche)

References

(Blocks 1 open bug)

Details

(Whiteboard: [measurement:client])

Attachments

(3 files)

We will need to show separate sections for the |scalars| and |keyedScalars| fields.
We can start this off as plain tables, filtering them etc. is not required.

This is not really blocked by other scalar work and could just be implemented based on test data.
Priority: -- → P3
Assignee: nobody → gfritzsche
Status: NEW → ASSIGNED
This adds trivial rendering of the scalar measurements as a key-value-table. Lets defer adding a keyed scalar section to bug 1277806.
Attachment #8761525 - Flags: review?(alessio.placitelli)
This moves less commonly used about:telemetry sections further down to make the page more convenient to use.
Attachment #8761527 - Flags: review?(alessio.placitelli)
Priority: P3 → P1
Points: --- → 1
Comment on attachment 8761525 [details] [diff] [review]
Update about:telemetry to show scalar measurements

Review of attachment 8761525 [details] [diff] [review]:
-----------------------------------------------------------------

::: toolkit/content/aboutTelemetry.js
@@ +1552,5 @@
>  
> +var Scalars = {
> +  /**
> +   * Render the scalar data - if present - from the payload in a simple key-value table.
> +   * @param aPing A payload object to render the data from.

nit: aPing -> aPayload
Attachment #8761525 - Flags: review?(alessio.placitelli) → review+
Attachment #8761527 - Flags: review?(alessio.placitelli) → review+
Pushed by georg.fritzsche@googlemail.com:
https://hg.mozilla.org/integration/fx-team/rev/2d81d52f4039
Update about:telemetry to show scalar measurements. r=dexter
https://hg.mozilla.org/integration/fx-team/rev/ce7f534b2580
Bonus: re-order main ping sections in about:telemetry by importance. r=dexter
https://hg.mozilla.org/mozilla-central/rev/2d81d52f4039
https://hg.mozilla.org/mozilla-central/rev/ce7f534b2580
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.