1486901 - Browser restarts should use launcher process when --enable-launcher-process is set

Status: RESOLVED FIXED

(Firefox :: General, enhancement, P1)

Description

[(No longer employed by Mozilla) Aaron Klotz]

Things like safe mode restarts, restarting from the profile manager, etc.

Comment 1

[(No longer employed by Mozilla) Aaron Klotz]

Attachment: Bug 1486901: Part 1 - Allow RunAsLauncherProcess to fall back to environment and command line if the parent process check does not return true; r=mhowell!

Comment 2

[(No longer employed by Mozilla) Aaron Klotz]

Attachment: Bug 1486901: Part 2 - Set MOZ_LAUNCHER_PROCESS when the launcher process is enabled by default and restarting via LaunchChild; r=mhowell!

This patch causes Firefox processes created via LaunchChild to start via the
launcher process.

While this extra hop through the launcher process takes longer than if we were
to simply make LaunchChild use the same code as the launcher process itself,
I do not want to simply export LauncherProcessMain from firefox.exe, as that
would be too easy to hook into and mess with.

Another option would be to host a second copy of the launcher code inside
xul.dll, but that's less than ideal.

Anyway, until we get that figured out, I don't think it's too big a deal to
have this extra hop in place. I don't think anybody has been clamoring for
faster restarts from the updater or the profile manager.

Depends on D4496

Comment 3

[Molly Howell (she/her) (mostly inactive)]

Comment on attachment 9004711 [details]
Bug 1486901: Part 1 - Allow RunAsLauncherProcess to fall back to environment and command line if the parent process check does not return true; r=mhowell!

Matt Howell [:mhowell] has approved the revision.

Comment 4

[Molly Howell (she/her) (mostly inactive)]

Comment on attachment 9004712 [details]
Bug 1486901: Part 2 - Set MOZ_LAUNCHER_PROCESS when the launcher process is enabled by default and restarting via LaunchChild; r=mhowell!

Matt Howell [:mhowell] has approved the revision.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6f6c28eb6041
Part 1 - Allow RunAsLauncherProcess to fall back to environment and command line if the parent process check does not return true; r=mhowell!

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/511ad3e51be3
Part 2 - Set MOZ_LAUNCHER_PROCESS when the launcher process is enabled by default and restarting via LaunchChild; r=mhowell!

Comment 7

[Eliza Balazs [:ebalazs_]]

https://hg.mozilla.org/mozilla-central/rev/6f6c28eb6041
https://hg.mozilla.org/mozilla-central/rev/511ad3e51be3

Comment 8

[(No longer employed by Mozilla) Aaron Klotz]

In the wee hours of last night, I realized that there is a bug in this code. I need to land a follow-up patch.

Comment 9

[(No longer employed by Mozilla) Aaron Klotz]

Attachment: Bug 1486901: Part 3 - Ensure that all RunAsLauncherProcess tests run to ensure side effects; r=mhowell!

Comment 10

[Molly Howell (she/her) (mostly inactive)]

Comment on attachment 9004978 [details]
Bug 1486901: Part 3 - Ensure that all RunAsLauncherProcess tests run to ensure side effects; r=mhowell!

Matt Howell [:mhowell] has approved the revision.

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/cf15e6faf983
Part 3 - Ensure that all RunAsLauncherProcess tests run to ensure side effects; r=mhowell!

Comment 12

[Arthur Iakab [arthur_iakab]]

https://hg.mozilla.org/mozilla-central/rev/cf15e6faf983

Comment 13

[(No longer employed by Mozilla) Aaron Klotz]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=40fa81f5bec157292dea086bd1b7752420f514a9