1085168 - [EME] Set timeout on async GMP shutdown in non xpcom-shutdown case

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

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

We allow GMPs to shutdown asynchronously, but we only actually set a timeout on shut if we're shutting down in XPCOM shutdown. We should also set a timeout if we're shutting down in the normal plugin shutdown case too.

Comment 1

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

Attachment: Patch: Move GMPAsyncShutdown timeout into GMPParent class

Move the GMPAsyncShutdown timeout into the GMPParent class, so that if we do async shutdown in the non-xpcom shutdown case, we still have a timeout.

I hit this case while writing unit tests in but 1055395.

Comment 2

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

https://hg.mozilla.org/integration/mozilla-inbound/rev/1a7fc6ba2aa7

Comment 3

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/1a7fc6ba2aa7

Comment 4

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

Mass update firefox-status to track EME uplift.