We need re-add ProcessRuntime so mscom is properly init'd. This was removed during one of the early reviews.

This fixes a CoInitializeEx failure when enabling Vorbis
decoding on RDD.

Just a note, without this fix, CoInitializeEx in RemoteDecoderManagerParent.cpp returns E_OUTOFMEMORY. With this fix, we get a different error (also a CoInitializeEx failure, but no win ApartmentRegion.h) which is reported in Bug 1553249.

https://hg.mozilla.org/mozilla-central/rev/e1d251e5e26f

Comment on attachment 9070715 [details]
Bug 1557844 - add back ProcessRuntime in RDD for mscom init. r?jya

ESR Uplift Approval Request

• If this is not a sec:{high,crit} bug, please state case for ESR consideration: Tor is built on ESR; and this patch is necessary for the RDD process to work in the MinGW build (which Tor uses)
• User impact if declined: Tor will need to carry a separate patch
• Fix Landed on Version: 69
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): I believe this patch is not risky, even though it applies to all platforms, because it is a bugfix needed for normal RDD process operation for Vorbis, just not AV1 (on clang-cl builds). But I'm going to confirm this.
• String or UUID changes made by this patch:

Michael, do you have any concerns about this patch landing in ESR 68?

This should be low risk.

Comment on attachment 9070715 [details]
Bug 1557844 - add back ProcessRuntime in RDD for mscom init. r?jya

Fixes an RDD issue affecting Tor browser. No issues with the patch on Beta so far. Approved for 68.1esr.

https://hg.mozilla.org/releases/mozilla-esr68/rev/2160cc9168ad