Closed Bug 1335859 Opened 7 years ago Closed 7 years ago

Add usage counters to track usage of gecko-only DataTransfer APIs

Categories

(Core :: DOM: Events, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla54
Tracking Status
firefox54 --- fixed

People

(Reporter: nika, Assigned: nika)

References

Details

Attachments

(1 file)

There is currently a decent amount of complexity in the DataTransfer{Item{List,},} APIs due to the need to support both the standard APIs, and the moz* APIs (which expose a different backing storage mechanism). 

This patch adds usage counters to these gecko-only APIs such that we can track their usage, and if they drop enough, then potentially remove them from the codebase.
MozReview-Commit-ID: L3vQXvWotUs
Attachment #8832594 - Flags: review?(amarchesini)
Attachment #8832594 - Flags: review?(amarchesini) → review+
Comment on attachment 8832594 [details] [diff] [review]
Add usage counters to track usage of gecko-only DataTransfer APIs

This adds telemetry, so asking for a data collection review.
Attachment #8832594 - Flags: feedback?(benjamin)
Comment on attachment 8832594 [details] [diff] [review]
Add usage counters to track usage of gecko-only DataTransfer APIs

Since use counters currently don't have expiration rules, please make sure a bug is on file to remove these when you no longer need them. data-r=me
Attachment #8832594 - Flags: feedback?(benjamin) → feedback+
Pushed by michael@thelayzells.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a8ef6f524ce7
Add usage counters to track usage of gecko-only DataTransfer APIs, r=baku
https://hg.mozilla.org/mozilla-central/rev/a8ef6f524ce7
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: