Add Telemetry probes for windows which have had mutation event listeners

RESOLVED FIXED

Status

()

Core
DOM
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: smaug, Assigned: smaug)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

2.12 KB, patch
(dormant account)
: review+
Details | Diff | Splinter Review
(Assignee)

Description

6 years ago
We could count how often documents have mutation event listeners.
If they are being used a lot, it may take some more time to get rid of
mutation events.
(Assignee)

Comment 1

6 years ago
So the outcome would be something like y of x created documents had mutation event listeners.
(Assignee)

Updated

6 years ago
Summary: Add Telemetry probes for mutation event listeners count → Add Telemetry probes for windows which have had mutation event listeners
(Assignee)

Comment 2

6 years ago
Created attachment 566788 [details] [diff] [review]
patch

So I want to count how many (inner) windows have had mutation event listeners
and how many hasn't had.
Based on what about:telemetry reports, this does that.
Assignee: nobody → Olli.Pettay
Status: NEW → ASSIGNED
Attachment #566788 - Flags: review?(tglek)
(Assignee)

Comment 3

6 years ago
Created attachment 566812 [details] [diff] [review]
patch

Just a bit nicer coding style.
Attachment #566788 - Attachment is obsolete: true
Attachment #566788 - Flags: review?(tglek)
Attachment #566812 - Flags: review?(tglek)

Updated

6 years ago
Attachment #566812 - Flags: review?(tglek) → review+

Updated

6 years ago
Keywords: privacy-review-needed
Can we please also let devs know that mutation events will get deprecated at some point? Even if we don't have a replacement for them yet, more information seems better than less. We can even provide an eta of when replacement will be available.
(Assignee)

Comment 5

6 years ago
Sure, but that has nothing to do with this bug :)
This is performance and usage-related, so fine privacy-wise.  Clearing the flag.
Keywords: privacy-review-needed
(Assignee)

Comment 7

6 years ago
https://hg.mozilla.org/mozilla-central/rev/45ac18984273
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.