GitLab

at 2023-02-23T09:54:29+01:00

fixup! Base Browser's .mozconfigs.

Bug 41629: Set MOZ_SERVICES_SYNC to False

Bug 41629: Fix errors with MOZ_SERVICES_SYNC=False

MOZ_SERVICES_SYNC should have been removed in Moz Bug 1227361.
However, it is still available, so we would like to use it.
Since it is a configuration that Mozilla does not test, and using it
results in a build error and in a few runtime errors, too.
This commit fixes them.

We have an upstream bug, too, but its (proposed) fix does not apply to
ESR 102 because of ESMification.
Should it not be accepted, we could replace this commit with the
poposed fix.
Upstream: https://bugzilla.mozilla.org/show_bug.cgi?id=1816969

 // This file is loaded into the browser window scope.

 /* eslint-env mozilla/browser-window */

-const { UIState } = ChromeUtils.import("resource://services-sync/UIState.jsm");

+ChromeUtils.defineModuleGetter(

+  this,

+  "UIState",

+  "resource://services-sync/UIState.jsm"

+);

 ChromeUtils.defineModuleGetter(

   this,

   // once syncing completes (bug 1239042).

   _syncStartTime: 0,

   _syncAnimationTimer: 0,

-  _obs: ["weave:engine:sync:finish", "quit-application", UIState.ON_UPDATE],

+  _obs: AppConstants.MOZ_SERVICES_SYNC

+    ? ["weave:engine:sync:finish", "quit-application", UIState.ON_UPDATE]

+    : [],

   get log() {

     if (!this._log) {

     this._definePrefGetters();

-    if (!this.FXA_ENABLED) {

+    if (!AppConstants.MOZ_SERVICES_SYNC || !this.FXA_ENABLED) {

       this.onFxaDisabled();

       return;

     }

     // can lead to a empty label for 'Send To Device' Menu.

     this.init();

-    if (!this.FXA_ENABLED) {

+    if (!AppConstants.MOZ_SERVICES_SYNC || !this.FXA_ENABLED) {

       // These items are hidden in onFxaDisabled(). No need to do anything.

       return;

     }

   // "Send Page to Device" and "Send Link to Device" menu items

   updateContentContextMenu(contextMenu) {

-    if (!this.FXA_ENABLED) {

+    if (!AppConstants.MOZ_SERVICES_SYNC || !this.FXA_ENABLED) {

       // These items are hidden by default. No need to do anything.

       return false;

     }

   };

 }

-XPCOMUtils.defineLazyGetter(

-  this,

-  "WeaveService",

-  () => Cc["@mozilla.org/weave/service;1"].getService().wrappedJSObject

+XPCOMUtils.defineLazyGetter(this, "WeaveService", () =>

+  AppConstants.MOZ_SERVICES_SYNC

+    ? Cc["@mozilla.org/weave/service;1"].getService().wrappedJSObject

+    : null

 );

 if (AppConstants.MOZ_CRASHREPORTER) {

       // Schedule a sync (if enabled) after we've loaded

       {

         task: async () => {

-          if (WeaveService.enabled) {

+          if (WeaveService?.enabled) {

             await WeaveService.whenLoaded();

             WeaveService.Weave.Service.scheduler.autoConnect();

           }

 @RESPATH@/browser/components/MacTouchBar.manifest

 @RESPATH@/browser/components/MacTouchBar.js

 #endif

+; TODO: Remove this in ESR-115.

+; If everything goes well, this patch will not be necessary in 115, because we

+; have also an upstream bug.

+; I suspect this is somehow incorrect, and that MOZ_SERVICES_SYNC is actually

+; never defined for the makefile (it is not for Firefox 112, which builds

+; correctly with MOZ_SERVICES_SYNC == False, even without this ifdef).

+; But we are interested in disabling it, so using either this, or #if 0 would be

+; fine for us.

+#ifdef MOZ_SERVICES_SYNC

 @RESPATH@/components/SyncComponents.manifest

+#endif

 @RESPATH@/components/servicesComponents.manifest

 @RESPATH@/components/servicesSettings.manifest

 @RESPATH@/components/cryptoComponents.manifest

 imply_option("MOZ_PLACES", True)

 # tor-browser#32493

 imply_option("MOZ_SERVICES_HEALTHREPORT", False)

-imply_option("MOZ_SERVICES_SYNC", True)

+# tor-browser#41629

+imply_option("MOZ_SERVICES_SYNC", False)

 imply_option("MOZ_DEDICATED_PROFILES", False)

 imply_option("MOZ_BLOCK_PROFILE_DOWNGRADE", False)

 # tor-browser#33734

 for var in (

     "MOZ_ALLOW_ADDON_SIDELOAD",

     "MOZ_BACKGROUNDTASKS",

+    "MOZ_SERVICES_SYNC",

     "MOZ_SYSTEM_NSS",

     "MOZ_SYSTEM_POLICIES",

     "MOZ_UNSIGNED_APP_SCOPE",

Pier Angelo Vendrame pushed to branch tor-browser-102.8.0esr-12.5-1 at The Tor Project / Applications / Tor Browser

Commits:

5 changed files:

Changes: