corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1596777 - Hook up the SecondaryVideoContainer with state mirroring. r=alwu

Browse Source 
This better follows how MediaDecoder and MediaDecoderStateMachine was
architected.

Differential Revision: https://phabricator.services.mozilla.com/D53710
This commit is contained in:
Andreas Pehrson
2019-11-20 16:32:52 +00:00
parent 0688763911
commit 2029e8192f
5 changed files with 44 additions and 40 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
dom/html/HTMLVideoElement.cpp
View File

@@ -502,12 +502,10 @@ void HTMLVideoElement::MaybeBeginCloningVisually() {
}
if (mDecoder) {
MediaDecoderStateMachine* mdsm = mDecoder->GetStateMachine();
VideoFrameContainer* container =
mVisualCloneTarget->GetVideoFrameContainer();
if (mdsm && container) {
mdsm->SetSecondaryVideoContainer(container);
mDecoder->SetCloningVisually(true);
if (container) {
mDecoder->SetSecondaryVideoContainer(container);
}
} else if (mSrcStream) {
VideoFrameContainer* container =
@@ -522,11 +520,7 @@ void HTMLVideoElement::EndCloningVisually() {
MOZ_ASSERT(mVisualCloneTarget);
if (mDecoder) {
MediaDecoderStateMachine* mdsm = mDecoder->GetStateMachine();
if (mdsm) {
mdsm->SetSecondaryVideoContainer(nullptr);
mDecoder->SetCloningVisually(false);
}
mDecoder->SetSecondaryVideoContainer(nullptr);
} else if (mSrcStream) {
VideoFrameContainer* container =
mVisualCloneTarget->GetVideoFrameContainer();