Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-10-10 03:45:46 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Bug 1540587 - remove patch for 1539030 to fix decoding on additional tabs. r=jya

Browse Source 
The patch for 1539030 didn't do what I'd hoped, and caused issues
with decoding av1 on additional tabs so I'm removing the "fix".

Differential Revision: https://phabricator.services.mozilla.com/D25627

--HG--
extra : moz-landing-system : lando
This commit is contained in:
Michael Froman 2019-04-03 03:12:35 +00:00
parent fcac8589b7
commit 23cebd517c
2 changed files with 1 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
dom/ipc/ContentChild.cpp
View File

@ -1173,9 +1173,7 @@ void ContentChild::LaunchRDDProcess() {
nsresult rv;
Endpoint<PRemoteDecoderManagerChild> endpoint;
Unused << SendLaunchRDDProcess(&rv, &endpoint);
// Only call InitForContent if we got a valid enpoint back which
// indicates we needed to launch an RDD process.
if (rv == NS_OK && endpoint.IsValid()) {
if (rv == NS_OK) {
RemoteDecoderManagerChild::InitForContent(std::move(endpoint));
}
}));

8
dom/ipc/ContentParent.cpp
View File

@ -1068,14 +1068,6 @@ mozilla::ipc::IPCResult ContentParent::RecvLaunchRDDProcess(
Preferences::GetBool("media.rdd-process.enabled", false)) {
RDDProcessManager* rdd = RDDProcessManager::Get();
if (rdd) {
// If there is already an RDDChild, then we've already launched the
// RDD process. We don't need to do anything else. Specifically,
// we want to avoid calling CreateContentBridge again because that
// causes the RemoteDecoderManagerParent to rebuild needlessly.
if (rdd->GetRDDChild()) {
return IPC_OK();
}
rdd->LaunchRDDProcess();
bool rddOpened = rdd->CreateContentBridge(OtherPid(), aEndpoint);