Closed Bug 1477790 Opened 4 years ago Closed 4 years ago

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

Categories

(Firefox :: General, enhancement, P1)

Unspecified
Windows
enhancement

Tracking

()

RESOLVED FIXED
Firefox 63
Tracking Status
firefox63 --- fixed

People

(Reporter: bugzilla, Assigned: bugzilla)

References

(Blocks 1 open bug)

Details

(Whiteboard: inj+)

Attachments

(1 file, 1 obsolete file)

No description provided.
Depends on: 1481549
Depends on: 1481635
I made a change to the launcher's DLL Blocklist so that those definitions are included when launcher by default is configured.
Attachment #8998361 - Attachment is obsolete: true
Attachment #8998361 - Flags: review?(mhowell)
Attachment #8998384 - Flags: review?(mhowell)
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.
Attachment #8998384 - Flags: review?(mhowell) → review+
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
https://hg.mozilla.org/mozilla-central/rev/52ae6b5241f8
https://hg.mozilla.org/mozilla-central/rev/51be350d0241
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
Regressions: 1576334
No longer regressions: 1576334
You need to log in before you can comment on or make changes to this bug.