Closed Bug 1167292 Opened 5 years ago Closed 4 years ago

Add telemetry for heap snapshots

Categories

(DevTools :: Memory, defect)

defect
Not set

Tracking

(firefox42 fixed)

RESOLVED FIXED
Firefox 42
Tracking Status
firefox42 --- fixed

People

(Reporter: fitzgen, Assigned: fitzgen)

References

(Blocks 1 open bug)

Details

Attachments

(3 files, 2 obsolete files)

* Time to take a snapshot

* Number of nodes in the heap graph

* Number of edges in the heap graph
Assignee: nobody → nfitzgerald
Status: NEW → ASSIGNED
Attachment #8617006 - Flags: review?(jimb)
Attachment #8617007 - Flags: review?(jimb)
Attachment #8617008 - Flags: review?(jimb)
Comment on attachment 8617014 [details] [diff] [review]
Part 3: Add telemetry probes for node and edge counts when saving heap snapshots

Updated the max values these new histograms can accept after I did some tests with various tabs open.
Attachment #8617014 - Flags: review?(jimb)
Attachment #8617006 - Flags: review?(jimb) → review?(ejpbruel)
Attachment #8617007 - Flags: review?(jimb) → review?(ejpbruel)
Attachment #8617014 - Flags: review?(jimb) → review?(ejpbruel)
Attachment #8617006 - Flags: review?(ejpbruel) → review+
Attachment #8617007 - Flags: review?(ejpbruel) → review+
Comment on attachment 8617014 [details] [diff] [review]
Part 3: Add telemetry probes for node and edge counts when saving heap snapshots

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

Trivial patches, so not much to add.
Attachment #8617014 - Flags: review?(ejpbruel) → review+
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.