Closed Bug 1050521 Opened 7 years ago Closed 3 years ago
Make GMPService init async
Bug 991037 comment #39: --- Comment #39 from Byron Campen [:bwc] <firstname.lastname@example.org> 2014-08-06 13:36:11 PDT --- (In reply to Martin Thomson [:mt] from comment #38) > @@ +492,5 @@ > > + return false; > > + } > > + > > + // presumes that all GMP dir scans have been queued for the GMPThread > > + thread->Dispatch(WrapRunnableNM(&GMPReady), NS_DISPATCH_NORMAL); > > I think that the idea of relying on the side-effects of do_GetService above > makes me very nervous. What if the init of GMP requires a dispatch to > another thread to complete? > > Can't you modify the API of the GMP service to provide a notification when > it is done? That's more work, but seems less fragile. >
This bug is necessary for basic playback of EME video on Windows.
Component: Audio/Video: MediaStreamGraph → Audio/Video: GMP
Is this still relevant?
Priority: -- → P2
We can probably get away with WONTFIXing this for now, and reopen/refile if we need to. What we have now is working OK.
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.