Backed out 3 changesets (bug 1812275, bug 1812035, bug 1806501) for bc failures on browser_aboutprocesses_default_options.js .

Backed out changeset 4f13d8470d97 (bug 1806501)
Backed out changeset a276a2939cee (bug 1812275)
Backed out changeset 67775732c1a8 (bug 1812035)
This commit is contained in:
Narcis Beleuzu 2023-01-27 17:14:57 +02:00
Родитель 422f88afde
Коммит 9e96e91239
10 изменённых файлов: 13 добавлений и 54 удалений

Просмотреть файл

@ -2855,14 +2855,6 @@ BrowserGlue.prototype = {
},
},
{
// Starts the JSOracle process for ORB JavaScript validation, if it hasn't started already.
name: "start-orb-javascript-oracle",
task: () => {
ChromeUtils.ensureJSOracleStarted();
},
},
{
name: "browser-startup-idle-tasks-finished",
task: () => {

Просмотреть файл

@ -6,7 +6,6 @@
#include "ChromeUtils.h"
#include "JSOracleParent.h"
#include "js/CharacterEncoding.h"
#include "js/Object.h" // JS::GetClass
#include "js/PropertyAndElement.h" // JS_DefineProperty, JS_DefinePropertyById, JS_Enumerate, JS_GetProperty, JS_GetPropertyById, JS_SetProperty, JS_SetPropertyById, JS::IdVector
@ -1763,16 +1762,4 @@ bool ChromeUtils::IsDarkBackground(GlobalObject&, Element& aElement) {
double ChromeUtils::DateNow(GlobalObject&) { return JS_Now() / 1000.0; }
/* static */
void ChromeUtils::EnsureJSOracleStarted(GlobalObject&) {
JSOracleParent::WithJSOracle(
[](JSOracleParent* aParent) { MOZ_DIAGNOSTIC_ASSERT(aParent); });
}
/* static */
unsigned ChromeUtils::AliveUtilityProcesses(const GlobalObject&) {
const auto& utilityProcessManager =
mozilla::ipc::UtilityProcessManager::GetIfExists();
return utilityProcessManager ? utilityProcessManager->AliveProcesses() : 0;
}
} // namespace mozilla::dom

Просмотреть файл

@ -284,10 +284,6 @@ class ChromeUtils {
static bool IsDarkBackground(GlobalObject&, Element&);
static double DateNow(GlobalObject&);
static void EnsureJSOracleStarted(GlobalObject&);
static unsigned AliveUtilityProcesses(const GlobalObject&);
};
} // namespace dom

Просмотреть файл

@ -653,17 +653,6 @@ partial namespace ChromeUtils {
* Returns whether the background of the element is dark.
*/
boolean isDarkBackground(Element element);
/**
* Starts the JSOracle process for ORB JavaScript validation, if it hasn't started already.
*/
undefined ensureJSOracleStarted();
/**
* The number of currently alive utility processes.
*/
[ChromeOnly]
readonly attribute unsigned long aliveUtilityProcesses;
};
/*

Просмотреть файл

@ -130,8 +130,6 @@ class UtilityProcessManager final : public UtilityProcessHost::Listener {
// Shutdown all utility processes
void CleanShutdownAllProcesses();
uint16_t AliveProcesses();
private:
~UtilityProcessManager();
@ -198,6 +196,7 @@ class UtilityProcessManager final : public UtilityProcessHost::Listener {
RefPtr<ProcessFields> GetProcess(SandboxingKind);
bool NoMoreProcesses();
uint16_t AliveProcesses();
};
} // namespace ipc

Просмотреть файл

@ -24,9 +24,7 @@ async function crashDecoder(expectation) {
const audioPid = await getAudioDecoderPid(expectation);
ok(audioPid > 0, `Found an audio decoder ${audioPid}`);
const actorIsAudioDecoder = actorNames => {
return actorNames
.split(",")
.some(actorName => actorName.trim().startsWith("audio-decoder-"));
return actorNames.startsWith("audio-decoder-");
};
info(`Crashing audio decoder ${audioPid}`);
await crashSomeUtility(audioPid, actorIsAudioDecoder);

Просмотреть файл

@ -132,7 +132,7 @@
socketProcessRunning = 1;
}
let m2 = (vsizes.length == (4 + socketProcessRunning + ChromeUtils.aliveUtilityProcesses) &&
let m2 = (vsizes.length == (4 + socketProcessRunning) &&
endOfBrowsers.length == 3);
ok(m2, "three content processes present in loaded data");
good = good && !!m2;

Просмотреть файл

@ -230,7 +230,7 @@
if (SpecialPowers.Services.io.socketProcessLaunched) {
socketProcessRunning = 1;
}
is(aAmounts.length, (1 + socketProcessRunning + ChromeUtils.aliveUtilityProcesses),
ok(aAmounts.length == (1 + socketProcessRunning),
aName + " has " + aAmounts.length + " report");
let n = aAmounts[0];
if (!aCanBeUnreasonable) {

Просмотреть файл

@ -22,7 +22,7 @@
socketProcessRunning = 1;
}
let numToOpen = 3;
const expectedNumRemotes = numToOpen + socketProcessRunning + ChromeUtils.aliveUtilityProcesses;
const expectedNumRemotes = numToOpen + socketProcessRunning;
let numReady = 0;
// Create some remote processes, and set up message-passing so that
@ -89,7 +89,7 @@
} else {
ok(processes[i].startsWith("Browser (") || processes[i].startsWith("Web Content (") ||
(processes[i].startsWith("Socket (") && socketProcessRunning)
|| processes[i].startsWith("web (") || processes[i].startsWith("Utility ("),
|| processes[i].startsWith("web ("),
"correct non-empty process name prefix: " + processes[i]);
numNonEmptyProcesses++;
}

Просмотреть файл

@ -1870,15 +1870,13 @@ nsresult nsMemoryReporterManager::StartGettingReports() {
}
}
if (!IsRegistrationBlocked()) {
if (RefPtr<UtilityProcessManager> utility =
UtilityProcessManager::GetIfExists()) {
for (RefPtr<UtilityProcessParent>& parent :
utility->GetAllProcessesProcessParent()) {
if (RefPtr<MemoryReportingProcess> proc =
utility->GetProcessMemoryReporter(parent)) {
s->mChildrenPending.AppendElement(proc.forget());
}
if (RefPtr<UtilityProcessManager> utility =
UtilityProcessManager::GetIfExists()) {
for (RefPtr<UtilityProcessParent>& parent :
utility->GetAllProcessesProcessParent()) {
if (RefPtr<MemoryReportingProcess> proc =
utility->GetProcessMemoryReporter(parent)) {
s->mChildrenPending.AppendElement(proc.forget());
}
}
}