1790146 - Support media engine playback for PiP

Status: RESOLVED FIXED

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

Description

[Alastor Wu [:alwu]]

We currently don't support using PiP window for the media engine playback.

Comment 1

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:jimm, could you have a look please?

For more information, please visit auto_nag documentation.

Comment 2

[Alastor Wu [:alwu]]

Attachment: Bug 1790146 - part1 : move mSecondaryVideoContainer to the base class and perform base members cleanup in the base class.

Comment 3

[Alastor Wu [:alwu]]

Attachment: Bug 1790146 - part2 : remove redundant `mVideoFrameContainer` which already exists in the base class.

Depends on D162157

Comment 4

[Alastor Wu [:alwu]]

Attachment: Bug 1790146 - part3 : set image to the secondary container for the media engine playabck.

Depends on D162158

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/558f0315037f
part1 : move mSecondaryVideoContainer to the base class and perform base members cleanup in the base class. r=jolin
https://hg.mozilla.org/integration/autoland/rev/e6db18eefe24
part2 : remove redundant `mVideoFrameContainer` which already exists in the base class. r=jolin
https://hg.mozilla.org/integration/autoland/rev/030f538d8fe1
part3 : set image to the secondary container for the media engine playabck. r=jolin

Comment 6

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/558f0315037f
https://hg.mozilla.org/mozilla-central/rev/e6db18eefe24
https://hg.mozilla.org/mozilla-central/rev/030f538d8fe1