Open
Bug 1515972
Opened 6 years ago
Updated 2 years ago
Investigate if MediaEngineWebRTC::mMutex is still needed.
Categories
(Core :: WebRTC: Audio/Video, enhancement, P3)
Core
WebRTC: Audio/Video
Tracking
()
NEW
| Tracking | Status | |
|---|---|---|
| firefox66 | --- | affected |
People
(Reporter: jib, Unassigned)
Details
From review https://phabricator.services.mozilla.com/D14941#inline-82096
The mutex was added back in 2012 [1] pre-sandboxing. A code comment [2] still states this is needed due to "gUM runnables", but aren't we always on the media thread now?
[1] https://searchfox.org/mozilla-central/diff/bef0a4c1e20fe660b7ad40bb0f47d7bd721c3b6f/content/media/webrtc/MediaEngineWebRTC.h#241
[2] https://searchfox.org/mozilla-central/rev/2e5e28f518524f8af5b158ddb605022b6a2d68cf/dom/media/webrtc/MediaEngineWebRTC.h#78
|
||
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•