1477790 - Add the ability for firefox.exe to use the launcher process by default

Status: RESOLVED FIXED

(Firefox :: General, enhancement, P1)

Comment 1

[(No longer employed by Mozilla) Aaron Klotz]

Attachment: Run launcher by default when configured appropriately

Comment 2

[(No longer employed by Mozilla) Aaron Klotz]

Attachment: Run launcher by default when configured appropriately (r2)

I made a change to the launcher's DLL Blocklist so that those definitions are included when launcher by default is configured.

Comment 3

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

Comment on attachment 8998384 [details] [diff] [review]
Run launcher by default when configured appropriately (r2)

Review of attachment 8998384 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/app/winlauncher/LauncherProcessWin.cpp
@@ +144,4 @@
>  namespace mozilla {
>  
>  // Eventually we want to be able to set a build config flag such that, when set,
>  // this function will always return true.

This patch has both obsoleted and invalidated this comment by adding support for the build flag and also a condition even in the presence of the build flag.

Comment 4

[(No longer employed by Mozilla) Aaron Klotz]

https://hg.mozilla.org/integration/mozilla-inbound/rev/52ae6b5241f837abff6d93dd4774ae54a8a73a71
Bug 1477790: Add ability for firefox.exe to use launcher process by default when the build is configured to do so; r=mhowell

Comment 5

[(No longer employed by Mozilla) Aaron Klotz]

https://hg.mozilla.org/integration/mozilla-inbound/rev/51be350d0241b998491dd04a812474e65aa96497
Bug 1477790: Follow up - compensate for missing typedef in some SDKs; r=bustage

Comment 6

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/52ae6b5241f8
https://hg.mozilla.org/mozilla-central/rev/51be350d0241