1548686 - don't add RemoteDecoderModule in PDMFactory::CreatePDMs if e10s is off

Status: RESOLVED FIXED

(Core :: Audio/Video: Playback, defect, P2)

Comment 1

[Michael Froman [:mjf]]

When enabling Vorbis decoding on RDD, 1proc tests will fail because RDD will not return a decoder if not running on the content process. However, because Vorbis decoding on RDD is on, the AgnosticDecoder answer that it does not support Vorbis. We need to check to make sure e10s is on/off when deciding which decoder module (Agnostic or Remote) will support Vorbis.

Comment 2

[Michael Froman [:mjf]]

Attachment: Bug 1548686 - only add RemoteDecoderModule in CreatePDMs if e10s. r?jya

This can cause 1proc tests to fail because no decoder ends up supporting
a format. The particular case was enabling Vorbis decoding on RDD.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/514ed4bfb0bd
only add RemoteDecoderModule in CreatePDMs if e10s. r=jya

Comment 4

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/514ed4bfb0bd