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

Backed out 2 changesets (bug 1728331) for causing failures at RTCPeerConnection-videoDetectorTest.html. CLOSED TREE

Browse Source 
Backed out changeset b6649f0253c5 (bug 1728331)
Backed out changeset e9242af1224d (bug 1728331)
This commit is contained in:
Butkovits Atila
2022-03-29 00:04:47 +03:00
parent 37a3eef9d5
commit 520eb8752c
14 changed files with 361 additions and 172 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
docshell/base/CanonicalBrowsingContext.cpp
View File

@@ -1758,7 +1758,7 @@ void CanonicalBrowsingContext::PendingRemotenessChange::Clear() {
// When this PendingRemotenessChange was created, it was given a
// `mContentParent`.
if (mContentParent) {
mContentParent->RemoveKeepAlive(mTarget->BrowserId());
mContentParent->RemoveKeepAlive();
mContentParent = nullptr;
}
@@ -1914,7 +1914,7 @@ CanonicalBrowsingContext::ChangeRemoteness(
// Switching to local, so we don't need to create a new process, and will
// instead use our embedder process.
change->mContentParent = embedderBrowser->Manager();
change->mContentParent->AddKeepAlive(BrowserId());
change->mContentParent->AddKeepAlive();
change->ProcessLaunched();
return promise.forget();
}
@@ -1933,7 +1933,7 @@ CanonicalBrowsingContext::ChangeRemoteness(
aOptions.mReplaceBrowsingContext &&
aOptions.mRemoteType == existingProcess->GetRemoteType()) {
change->mContentParent = existingProcess;
change->mContentParent->AddKeepAlive(BrowserId());
change->mContentParent->AddKeepAlive();
change->ProcessLaunched();
return promise.forget();
}
@@ -1955,7 +1955,6 @@ CanonicalBrowsingContext::ChangeRemoteness(
change->mContentParent = ContentParent::GetNewOrUsedLaunchingBrowserProcess(
/* aRemoteType = */ aOptions.mRemoteType,
/* aGroup = */ finalGroup,
/* aBrowserId */ BrowserId(),
/* aPriority = */ hal::PROCESS_PRIORITY_FOREGROUND,
/* aPreferUsed = */ preferUsed);
if (!change->mContentParent) {
@@ -1966,7 +1965,7 @@ CanonicalBrowsingContext::ChangeRemoteness(
// Add a KeepAlive used by this ContentParent, which will be cleared when
// the change is complete. This should prevent the process dying before
// we're ready to use it.
change->mContentParent->AddKeepAlive(BrowserId());
change->mContentParent->AddKeepAlive();
if (change->mContentParent->IsLaunching()) {
change->mContentParent->WaitForLaunchAsync()->Then(
GetMainThreadSerialEventTarget(), __func__,