As an added bonus, this makes the temporal coupling between `setPrivateModeIfNecessary` and `setupThemeAndBrowsingMode` explicit. They previously would have broken if called in reverse order, now it will fail to compile.
This didn't function when 'open links in a private tab' was set. Rather than adding another sketchy fix for the edge case, following commits will change `usePrivateMode` to be maintained in memory, instead of in Settings.
It was fixed in [1], but I regressed it when I resolved conflicts in [2]
[1] 9729fd494e\#diff-3a2aaafc93fc8bb53e2029001aa236aeL98
[2] 060e915d2b\#diff-3a2aaafc93fc8bb53e2029001aa236aeR95
This patch enabled support for an auto-publication workflow for android-components.
It automates a common pattern seen in local development:
Old way:
- after every change in a-c, publish it locally with a unique version (bumping it manually)
- manually modify Fenix to consume a custom version of a-c from a mavenLocal repository
New way:
- set a flag in fenix's local.properties to enable auto-publication
- run Fenix builds after making changes to a-c. Changes in a-c will be automatically picked up.
Note that no changes are necessary to any Fenix files other than a single flag in local.properties.
Manually bumping android-components version is also not necessary.
* Do not launch in Private Mode
When the app launches do not launch in Private Mode in order to prevent usage leaks to other users of the device.
* Issue #4780: add comments to use private mode
* For #4780: write tests for clear private mode on create app
* For #4780: clear private mode when privacy notification is removed
Michael Droettboom
Mozilla L10n Automation Bot
Roger Yang
Tiger Oakes
Sawyer Blatz
Severin Rudie
Mitchell Hentges
Johan Lorenzo
Aaron Train
mcarare
Mihai Adrian
isabelrios
Isabel Rios
Grisha Kruglov
Vipul Asri
sv-ohorvath
Denys M
Madalin Valceleanu
ekager
Prat T
Emily Kager
Colin Lee
Christian Sadilek
Severin Rudie