1050521 - Make GMPService init async

Status: RESOLVED WONTFIX

(Core :: Audio/Video: GMP, defect, P3)

Description

[Chris Pearce [:cpearce (Not reading bugmail)]]

Bug 991037 comment #39:

--- Comment #39 from Byron Campen [:bwc] <[e-mail]> 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.
>

Comment 1

[Chris Peterson [:cpeterson]]

This bug is necessary for basic playback of EME video on Windows.

Comment 2

[Maire Reavy [:mreavy]]

Is this still relevant?

Comment 3

[Chris Pearce [:cpearce (Not reading bugmail)]]

We can probably get away with WONTFIXing this for now, and reopen/refile if we need to. What we have now is working OK.

Comment 4

[Bulk Bug Changes for mreavy's org]

Mass change P2->P3 to align with new Mozilla triage process.