Commits:
-
bc8fc896
by Richard Pospesel at 2023-02-27T09:47:57+01:00
Revert "Bug 41629: Fix errors with MOZ_SERVICES_SYNC=False"
This reverts commit 8f61c9c0b06a1820caeb4683595cec98f0ad88b9.
current implementation breaks about:preferences due to runtime errors
-
a4aaf6e6
by Richard Pospesel at 2023-02-27T09:48:26+01:00
Revert "fixup! Base Browser's .mozconfigs."
This reverts commit 9ac0abfdd9f4bcb53841c40d3b42d34bec92b262.
5 changed files:
Changes:
browser/base/content/browser-sync.js
| ... |
... |
@@ -5,11 +5,7 @@ |
|
5
|
5
|
// This file is loaded into the browser window scope.
|
|
6
|
6
|
/* eslint-env mozilla/browser-window */
|
|
7
|
7
|
|
|
8
|
|
-ChromeUtils.defineModuleGetter(
|
|
9
|
|
- this,
|
|
10
|
|
- "UIState",
|
|
11
|
|
- "resource://services-sync/UIState.jsm"
|
|
12
|
|
-);
|
|
|
8
|
+const { UIState } = ChromeUtils.import("resource://services-sync/UIState.jsm");
|
|
13
|
9
|
|
|
14
|
10
|
ChromeUtils.defineModuleGetter(
|
|
15
|
11
|
this,
|
| ... |
... |
@@ -346,9 +342,7 @@ var gSync = { |
|
346
|
342
|
// once syncing completes (bug 1239042).
|
|
347
|
343
|
_syncStartTime: 0,
|
|
348
|
344
|
_syncAnimationTimer: 0,
|
|
349
|
|
- _obs: AppConstants.MOZ_SERVICES_SYNC
|
|
350
|
|
- ? ["weave:engine:sync:finish", "quit-application", UIState.ON_UPDATE]
|
|
351
|
|
- : [],
|
|
|
345
|
+ _obs: ["weave:engine:sync:finish", "quit-application", UIState.ON_UPDATE],
|
|
352
|
346
|
|
|
353
|
347
|
get log() {
|
|
354
|
348
|
if (!this._log) {
|
| ... |
... |
@@ -464,7 +458,7 @@ var gSync = { |
|
464
|
458
|
|
|
465
|
459
|
this._definePrefGetters();
|
|
466
|
460
|
|
|
467
|
|
- if (!AppConstants.MOZ_SERVICES_SYNC || !this.FXA_ENABLED) {
|
|
|
461
|
+ if (!this.FXA_ENABLED) {
|
|
468
|
462
|
this.onFxaDisabled();
|
|
469
|
463
|
return;
|
|
470
|
464
|
}
|
| ... |
... |
@@ -1535,7 +1529,7 @@ var gSync = { |
|
1535
|
1529
|
// can lead to a empty label for 'Send To Device' Menu.
|
|
1536
|
1530
|
this.init();
|
|
1537
|
1531
|
|
|
1538
|
|
- if (!AppConstants.MOZ_SERVICES_SYNC || !this.FXA_ENABLED) {
|
|
|
1532
|
+ if (!this.FXA_ENABLED) {
|
|
1539
|
1533
|
// These items are hidden in onFxaDisabled(). No need to do anything.
|
|
1540
|
1534
|
return;
|
|
1541
|
1535
|
}
|
| ... |
... |
@@ -1570,7 +1564,7 @@ var gSync = { |
|
1570
|
1564
|
|
|
1571
|
1565
|
// "Send Page to Device" and "Send Link to Device" menu items
|
|
1572
|
1566
|
updateContentContextMenu(contextMenu) {
|
|
1573
|
|
- if (!AppConstants.MOZ_SERVICES_SYNC || !this.FXA_ENABLED) {
|
|
|
1567
|
+ if (!this.FXA_ENABLED) {
|
|
1574
|
1568
|
// These items are hidden by default. No need to do anything.
|
|
1575
|
1569
|
return false;
|
|
1576
|
1570
|
}
|
browser/components/BrowserGlue.jsm
| ... |
... |
@@ -718,10 +718,10 @@ let JSWINDOWACTORS = { |
|
718
|
718
|
},
|
|
719
|
719
|
};
|
|
720
|
720
|
|
|
721
|
|
-XPCOMUtils.defineLazyGetter(this, "WeaveService", () =>
|
|
722
|
|
- AppConstants.MOZ_SERVICES_SYNC
|
|
723
|
|
- ? Cc["@mozilla.org/weave/service;1"].getService().wrappedJSObject
|
|
724
|
|
- : null
|
|
|
721
|
+XPCOMUtils.defineLazyGetter(
|
|
|
722
|
+ this,
|
|
|
723
|
+ "WeaveService",
|
|
|
724
|
+ () => Cc["@mozilla.org/weave/service;1"].getService().wrappedJSObject
|
|
725
|
725
|
);
|
|
726
|
726
|
|
|
727
|
727
|
if (AppConstants.MOZ_CRASHREPORTER) {
|
| ... |
... |
@@ -2690,7 +2690,7 @@ BrowserGlue.prototype = { |
|
2690
|
2690
|
// Schedule a sync (if enabled) after we've loaded
|
|
2691
|
2691
|
{
|
|
2692
|
2692
|
task: async () => {
|
|
2693
|
|
- if (WeaveService?.enabled) {
|
|
|
2693
|
+ if (WeaveService.enabled) {
|
|
2694
|
2694
|
await WeaveService.whenLoaded();
|
|
2695
|
2695
|
WeaveService.Weave.Service.scheduler.autoConnect();
|
|
2696
|
2696
|
}
|
browser/installer/package-manifest.in
| ... |
... |
@@ -179,17 +179,7 @@ |
|
179
|
179
|
@RESPATH@/browser/components/MacTouchBar.manifest
|
|
180
|
180
|
@RESPATH@/browser/components/MacTouchBar.js
|
|
181
|
181
|
#endif
|
|
182
|
|
-; TODO: Remove this in ESR-115.
|
|
183
|
|
-; If everything goes well, this patch will not be necessary in 115, because we
|
|
184
|
|
-; have also an upstream bug.
|
|
185
|
|
-; I suspect this is somehow incorrect, and that MOZ_SERVICES_SYNC is actually
|
|
186
|
|
-; never defined for the makefile (it is not for Firefox 112, which builds
|
|
187
|
|
-; correctly with MOZ_SERVICES_SYNC == False, even without this ifdef).
|
|
188
|
|
-; But we are interested in disabling it, so using either this, or #if 0 would be
|
|
189
|
|
-; fine for us.
|
|
190
|
|
-#ifdef MOZ_SERVICES_SYNC
|
|
191
|
182
|
@RESPATH@/components/SyncComponents.manifest
|
|
192
|
|
-#endif
|
|
193
|
183
|
@RESPATH@/components/servicesComponents.manifest
|
|
194
|
184
|
@RESPATH@/components/servicesSettings.manifest
|
|
195
|
185
|
@RESPATH@/components/cryptoComponents.manifest
|
browser/moz.configure
| ... |
... |
@@ -7,8 +7,7 @@ |
|
7
|
7
|
imply_option("MOZ_PLACES", True)
|
|
8
|
8
|
# tor-browser#32493
|
|
9
|
9
|
imply_option("MOZ_SERVICES_HEALTHREPORT", False)
|
|
10
|
|
-# tor-browser#41629
|
|
11
|
|
-imply_option("MOZ_SERVICES_SYNC", False)
|
|
|
10
|
+imply_option("MOZ_SERVICES_SYNC", True)
|
|
12
|
11
|
imply_option("MOZ_DEDICATED_PROFILES", True)
|
|
13
|
12
|
imply_option("MOZ_BLOCK_PROFILE_DOWNGRADE", True)
|
|
14
|
13
|
# tor-browser#33734
|
toolkit/modules/moz.build
| ... |
... |
@@ -291,7 +291,6 @@ for var in ( |
|
291
|
291
|
for var in (
|
|
292
|
292
|
"MOZ_ALLOW_ADDON_SIDELOAD",
|
|
293
|
293
|
"MOZ_BACKGROUNDTASKS",
|
|
294
|
|
- "MOZ_SERVICES_SYNC",
|
|
295
|
294
|
"MOZ_SYSTEM_NSS",
|
|
296
|
295
|
"MOZ_SYSTEM_POLICIES",
|
|
297
|
296
|
"MOZ_UNSIGNED_APP_SCOPE",
|
|