Closed Bug 721019 Opened 13 years ago Closed 13 years ago

[Page Thumbnails] Add telemetry probes

Categories

(Firefox :: Tabbed Browser, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 13

People

(Reporter: ttaubert, Assigned: ttaubert)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Add telemetry probes that track: * the time it took to capture a thumbnail * the time it took to store a thumbnail * number of cache hits/misses * ...
Blocks: 671038
Attached patch patch v1Splinter Review
Assignee: nobody → ttaubert
Attachment #593459 - Flags: review?(dietrich)
Status: NEW → ASSIGNED
Comment on attachment 593459 [details] [diff] [review] patch v1 Review of attachment 593459 [details] [diff] [review]: ----------------------------------------------------------------- ::: browser/components/thumbnails/PageThumbs.jsm @@ +76,5 @@ > * @param aWindow The DOM window to capture a thumbnail from. > * @return The newly created canvas containing the image data. > */ > capture: function PageThumbs_capture(aWindow) { > + let telemetryCaptureTime = Date.now(); you can just "new Date()" instead, here and elsewhere.
Attachment #593459 - Flags: review?(dietrich) → review+
Please move this to the Firefox section in TelemetryHistograms.h and use the FX prefix. "THUMBNAILS" is too generic.
Will do.
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 13
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: