1272066 - Telemetry is apparently not threadsafe.

Status: RESOLVED FIXED

(Core :: DOM: Workers, defect)

Comment 1

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/a3a8e4f9cd21

Comment 2

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Landed https://hg.mozilla.org/integration/mozilla-inbound/rev/a3a8e4f9cd21 to hopefully stop it from asserting until a better fix can land.

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/3937939773fc

Comment 4

[Andrea Marchesini [:baku]]

Is this a duplicate of bug 1258183 ?

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/a3a8e4f9cd21
https://hg.mozilla.org/mozilla-central/rev/3937939773fc

Comment 6

[Julian Seward [:jseward]]

(In reply to Andrea Marchesini (:baku) from comment #4)
> Is this a duplicate of bug 1258183 ?

Yes, I think it is.  Bug 1258183 may not entirely de-race Telemetry, but it is a
good start.  There may be followups depending on whether any remaining Telemetry
races are observed with TSan after bug 1258183 lands.

Comment 7

[Andrea Marchesini [:baku]]

Attachment: telemetry.patch

It seems that all the dependences are now resolved.
I would like to enable this telemetry ID in runnables.

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5a794352cf9c
Add telemetry for sync worker runnables, r=bkelly

Comment 9

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/5a794352cf9c