1646088 - Ensure content processes cannot shutdown during async process switches

Status: RESOLVED FIXED

(Core :: DOM: Content Processes, defect, P3)

Description

[Nika Layzell [:nika] (ooo, ni? for response)]

I think it is possible right now for a shutdown message to be sent to a content process between the time when CanonicalBrowsingContext begins async process launching, and when the process is actually used to create a new BrowserParent.

This bug tracks adding some mechanism to prevent the process from shutting down during that time.

Comment 1

[Nika Layzell [:nika] (ooo, ni? for response)]

Attachment: Bug 1646088 - Part 1: Keep processes alive during process switches,

Comment 2

[Nika Layzell [:nika] (ooo, ni? for response)]

Attachment: Bug 1646088 - Part 2: Create ScriptableCPInfo in the constructor,

Comment 3

[Nika Layzell [:nika] (ooo, ni? for response)]

Attachment: Bug 1646088 - Part 3: Add a test for slow process switches,

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/77d15266af3c
Part 1: Keep processes alive during process switches, r=kmag
https://hg.mozilla.org/integration/autoland/rev/ef746bdcbaf6
Part 2: Create ScriptableCPInfo in the constructor, r=kmag
https://hg.mozilla.org/integration/autoland/rev/8c4a24b91c88
Part 3: Add a test for slow process switches, r=kmag

Comment 5

[Alexandru Michis [:malexandru]]

Backed out 3 changesets for causing failures in test_multiple_nav_process_switches.

Backout link: https://hg.mozilla.org/integration/autoland/rev/2799d2f79290ca6a9d7982f9eeb3640dccf8b08e

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&group_state=expanded&searchStr=linux%2C18.04%2Cx64%2Cdebug%2Cmochitests%2Cwith%2Cfission%2Cenabled%2Ctest-linux1804-64%2Fdebug-mochitest-browser-chrome-fis-e10s-12%2Cm-fis%28bc12%29&tochange=2799d2f79290ca6a9d7982f9eeb3640dccf8b08e&fromchange=176ca18a05abffa73b5606f724a6054ae6582378

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=306682786&repo=autoland&lineNumber=13897

Comment 6

[Nika Layzell [:nika] (ooo, ni? for response)]

Attachment: Bug 1646088 - Part 4: Update browser_httpCrossOriginOpenerPolicy for keeping processes alive during switches,

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/1a4de0e50652
Part 1: Keep processes alive during process switches, r=kmag
https://hg.mozilla.org/integration/autoland/rev/20252e44605a
Part 2: Create ScriptableCPInfo in the constructor, r=kmag
https://hg.mozilla.org/integration/autoland/rev/b21844a2bdeb
Part 3: Add a test for slow process switches, r=kmag
https://hg.mozilla.org/integration/autoland/rev/f2d3b3bb9261
Part 4: Update browser_httpCrossOriginOpenerPolicy for keeping processes alive during switches, r=valentin

Comment 8

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/1a4de0e50652
https://hg.mozilla.org/mozilla-central/rev/20252e44605a
https://hg.mozilla.org/mozilla-central/rev/b21844a2bdeb
https://hg.mozilla.org/mozilla-central/rev/f2d3b3bb9261