Closed Bug 1295980 Opened 6 years ago Closed 6 years ago

Let's add telemetry to see if SharedWorkers are used

Categories

(Core :: DOM: Workers, defect)

50 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla51
Tracking Status
firefox51 --- fixed

People

(Reporter: baku, Assigned: baku)

Details

Attachments

(1 file)

No description provided.
Attached patch telemetry.patchSplinter Review
Attachment #8782002 - Flags: review?(bugs)
Attachment #8782002 - Flags: review?(bugs) → review?(bkelly)
oh, Shared, not Service :) I take this one.
Attachment #8782002 - Flags: review?(bkelly) → review?(bugs)
Comment on attachment 8782002 [details] [diff] [review]
telemetry.patch

and per IRC the reason for this is that Apple removed SharedWorkers and we want to know whether they are being used at all.
Attachment #8782002 - Flags: review?(bugs) → review+
Adding telemetry is a good thing, but just want to highlight we expect future use cases for SharedWorker from ServiceWorker.  Specifically, some applications want a stable global for some operations.  We are planning to expose SharedWorker from ServiceWorker so they can do that work there.
Pushed by amarchesini@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f9c695f0f368
Telemetry about the use of SharedWorkers, r=smaug
https://hg.mozilla.org/mozilla-central/rev/f9c695f0f368
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in before you can comment on or make changes to this bug.