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)
Core
DOM: Events
Tracking
()
RESOLVED
FIXED
mozilla54
Tracking | Status | |
---|---|---|
firefox54 | --- | fixed |
People
(Reporter: nika, Assigned: nika)
References
Details
Attachments
(1 file)
5.64 KB,
patch
|
baku
:
review+
benjamin
:
feedback+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Comment 1•7 years ago
|
||
MozReview-Commit-ID: L3vQXvWotUs
Attachment #8832594 -
Flags: review?(amarchesini)
Updated•7 years ago
|
Attachment #8832594 -
Flags: review?(amarchesini) → review+
Assignee | ||
Comment 2•7 years ago
|
||
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 3•7 years ago
|
||
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
Comment 5•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/a8ef6f524ce7
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox54:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in
before you can comment on or make changes to this bug.
Description
•