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)

x86_64
All
enhancement

Tracking

()

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.
Component: Audio/Video → Audio/Video: Playback
Sending this back to Maire.
Component: Audio/Video: Playback → Audio/Video: MSG/cubeb/GMP
Is this resolved? If not, what are the issues remaining?
Rank: 35
Flags: needinfo?(jib)
Priority: -- → P3
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)
Are we still relying on xpcom for events on the media threads in media code? See bug 1060738, comment 39.
Flags: needinfo?(jmathies)
Not a required change, obviously things are working as is.
Severity: normal → enhancement
Mass change P3->P4 to align with new Mozilla triage process.
Priority: P3 → P4
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.