1694036 - Fix use of delay pref when starting preallocated processes

Status: RESOLVED FIXED

(Core :: DOM: Content Processes, defect)

Description

[Randell Jesup [:jesup] (needinfo me)]

We have a pref for a delay before prestarting processes to avoid conflicts with pageload. However, this pref isn't working properly - we're allocating on Idle with no delay. Starting new processes during a page load can significantly impact pageload.

As a related item, it makes sense to separate this delay from a delay prestarting processes at browser startup, since the optimum delay for the two cases is certainly different.

Comment 1

[Randell Jesup [:jesup] (needinfo me)]

Attachment: Bug 1694036: Fix process-start delay and add pref for startup delay r=nika

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a308dfa27b39
Fix process-start delay and add pref for startup delay r=nika

Comment 3

[Sandor Molnar[:smolnar]]

Backed out for causing mochitest failures in browser_process_crash

Backout link:https://hg.mozilla.org/integration/autoland/rev/916497e295fe6c3b5e8082045add889fd555dff6

Push with failures

Failure log

Comment 4

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/97d252381ef2
Backed out changeset a308dfa27b39 for causing mochitest failures in browser_process_crash. CLOSED TREE

Comment 5

[Randell Jesup [:jesup] (needinfo me)]

Attachment: Bug 1694036: fix testing race condition is about:home restart simulation r=mconley

Depends on D105878

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1a29ecee881e
Fix process-start delay and add pref for startup delay r=nika
https://hg.mozilla.org/integration/autoland/rev/cd61de1bc04e
fix testing race condition is about:home restart simulation r=mconley

Comment 7

[Narcis Beleuzu [:NarcisB]]

Backed out for bc failures on browser_process_crash.js

Backout link: https://hg.mozilla.org/integration/autoland/rev/c75f7cfc909566afb42e68de48161daa3e0e15a6
Log link: https://treeherder.mozilla.org/logviewer?job_id=330957216&repo=autoland&lineNumber=3846

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/50536730e438
Fix process-start delay and add pref for startup delay r=nika
https://hg.mozilla.org/integration/autoland/rev/eca16d059b36
fix testing race condition is about:home restart simulation r=mconley

Comment 9

[Bogdan Tara[:bogdan_tara | bogdant]]

https://hg.mozilla.org/mozilla-central/rev/50536730e438
https://hg.mozilla.org/mozilla-central/rev/eca16d059b36