Closed Bug 1139752 Opened 7 years ago Closed 7 years ago

Fix about:telemetry to work with the unified pings

Categories

(Toolkit :: Telemetry, defect)

x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla39
Tracking Status
firefox39 --- fixed

People

(Reporter: vladan, Assigned: vladan)

References

Details

Attachments

(1 file)

about:telemetry doesn't load because of an exception "TypeError: value is null"

  renderBody: function KeyValueTable_renderBody(aTable, aMeasurements) {
    for (let [key, value] of Iterator(aMeasurements)) {
      // use .valueOf() to unbox Number, String, etc. objects
====> if ((typeof value == "object") && (typeof value.valueOf() == "object")) {
        value = RenderObject(value);
      }

I think this exception occurs when handling the "System Information" section
Attachment #8574090 - Flags: review?(nfroyd)
Attachment #8574090 - Flags: review?(nfroyd) → review?(dteller)
Comment on attachment 8574090 [details] [diff] [review]
aboutTelemetryNull.patch

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

This definitely makes sense.
Attachment #8574090 - Flags: review?(dteller) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/11282d8746ea
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → mozilla39
Duplicate of this bug: 1137545
You need to log in before you can comment on or make changes to this bug.