- Truncate descriptions
Activity
Trac:
Parent Ticket: #33656 (moved)Making a note here:
ed7faf1c2b88cd025596151f2ad97892891965a4
contains:diff --git a/toolkit/modules/AppConstants.jsm b/toolkit/modules/AppConstants.jsm index 109f1f94e3d3..bc679fb6f9f0 100644 --- a/toolkit/modules/AppConstants.jsm +++ b/toolkit/modules/AppConstants.jsm @@ -387,4 +387,11 @@ this.AppConstants = Object.freeze({ #else false, #endif + + TOR_BROWSER_UPDATE: +#ifdef TOR_BROWSER_UPDATE + true, +#else + false, +#endif
But
TOR_BROWSER_UPDATE
is used earlier in14df73ecfdd88db89f196cecfb934dab7e552969
. We should move this code block into the earlier patch.During the 68esr rebase, I squashed the patch for configuring
about:tor
as the default homepage into the branding patch.Unfortunately, that patch changes Fennec UI behavior. We shouldn't squash Gecko modifications with Fennec UI modifications. I opened #31983 (moved) for the Fennec UI change, so we can break that functionality into a separate patch and prevent the current inconsistency on Android.
Okay,
tor-browser-68.1.0esr-9.0-3
is a squashed branch that could be used for this ticket. I already started moving commits around, e.g. moving the ones needed for actually building code even before our mozconfig commit.We should think about more changes that would facilitate bisecting and readability of the branch. E.g. moving the updater/mar signing patches earlier, grouping Mozilla patches that belong together etc.
Something we could do while working on this bug, and making the patches less desktop vs. android, is to look for pieces we disabled on desktop but not on mobile, yet we would like to have them actually disabled on both. E.g. https://bugzilla.mozilla.org/show_bug.cgi?id=1503402 comes to mind where we just no build the webcompat extension on desktop but still do so on mobile and include it there.
Trac:
Parent: N/A to #33656 (moved)