Closed
Bug 1272066
Opened 8 years ago
Closed 8 years ago
Telemetry is apparently not threadsafe.
Categories
(Core :: DOM: Workers, defect)
Core
DOM: Workers
Tracking
()
RESOLVED
FIXED
mozilla52
Tracking | Status | |
---|---|---|
firefox52 | --- | fixed |
People
(Reporter: KWierso, Assigned: baku)
References
Details
(Whiteboard: btpp-active)
Attachments
(1 file)
1.21 KB,
patch
|
bkelly
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Reporter | ||
Comment 2•8 years ago
|
||
Landed https://hg.mozilla.org/integration/mozilla-inbound/rev/a3a8e4f9cd21 to hopefully stop it from asserting until a better fix can land.
Keywords: leave-open
Comment 5•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/a3a8e4f9cd21 https://hg.mozilla.org/mozilla-central/rev/3937939773fc
Comment 6•8 years ago
|
||
(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.
Flags: needinfo?(jseward)
Updated•8 years ago
|
Whiteboard: btpp-active
Assignee | ||
Comment 7•8 years ago
|
||
It seems that all the dependences are now resolved. I would like to enable this telemetry ID in runnables.
Attachment #8804203 -
Flags: review?(bkelly)
Updated•8 years ago
|
Attachment #8804203 -
Flags: review?(bkelly) → review+
Pushed by amarchesini@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/5a794352cf9c Add telemetry for sync worker runnables, r=bkelly
Assignee | ||
Updated•8 years ago
|
Keywords: leave-open
Comment 9•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/5a794352cf9c
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox52:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in
before you can comment on or make changes to this bug.
Description
•