Open
Bug 1066662
Opened 10 years ago
Updated 2 years ago
Remove remaining reliance on xpcom events for MediaManager background thread
Categories
(Core :: Audio/Video: MediaStreamGraph, enhancement, P4)
Tracking
()
NEW
People
(Reporter: jimm, Unassigned)
References
Details
Follow up work from bug 1060738 - according to jesup:
(In reply to Randell Jesup [:jesup] from comment #63)
> Note: The reason the Try in comment 32 failed was because
> MediaEngineDefault.cpp uses nsITimers on MediaManager thread to generate
> fake audio/video; these could easily be swapped or spun off to another
> thread.
If we can remove reliance on xpcom events, we can stop using franken message loop types TYPE_MOZILLA_NONMAINTHREAD (mac, linux) / TYPE_MOZILLA_NONMAINUITHREAD (win) and instead use just TYPE_DEFAULT/TYPE_UI.
Updated•9 years ago
|
Component: Audio/Video → Audio/Video: Playback
Sending this back to Maire.
Component: Audio/Video: Playback → Audio/Video: MSG/cubeb/GMP
Comment 2•9 years ago
|
||
Is this resolved? If not, what are the issues remaining?
Rank: 35
Flags: needinfo?(jib)
Priority: -- → P3
Comment 3•9 years ago
|
||
Sorry for sitting on this, but I don't fully comprehend this improvement request.
A quick look shows the things mentioned in comment 0 is still in use:
http://mxr.mozilla.org/mozilla-central/search?string=TYPE_MOZILLA_NONMAINTHREAD
Deflecting to jimm to answer.
Flags: needinfo?(jib) → needinfo?(jmathies)
Reporter | ||
Comment 4•9 years ago
|
||
Are we still relying on xpcom for events on the media threads in media code? See bug 1060738, comment 39.
Flags: needinfo?(jmathies)
Reporter | ||
Comment 5•9 years ago
|
||
Not a required change, obviously things are working as is.
Severity: normal → enhancement
Comment 6•7 years ago
|
||
Mass change P3->P4 to align with new Mozilla triage process.
Priority: P3 → P4
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•