Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

Commit b786539

Browse files
author
Mark Banner
committed
Bug 827303 - Firefox core and crypto services components assume services/sync is built. Move registrations out of services/sync. r=gps
--HG-- rename : services/sync/SyncComponents.manifest => services/common/servicesComponents.manifest rename : services/sync/SyncComponents.manifest => services/crypto/cryptoComponents.manifest
1 parent 41dec50 commit b786539

9 files changed

Lines changed: 22 additions & 3 deletions

File tree

b2g/installer/package-manifest.in

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -484,6 +484,8 @@
484484
@BINPATH@/components/WeaveCrypto.manifest
485485
@BINPATH@/components/WeaveCrypto.js
486486
#endif
487+
@BINPATH@/components/servicesComponents.manifest
488+
@BINPATH@/components/cryptoComponents.manifest
487489
#ifdef MOZ_SERVICES_HEALTHREPORT
488490
@BINPATH@/components/HealthReportComponents.manifest
489491
@BINPATH@/components/HealthReportService.js

browser/installer/package-manifest.in

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -477,6 +477,8 @@
477477
@BINPATH@/components/SyncComponents.manifest
478478
@BINPATH@/components/Weave.js
479479
#endif
480+
@BINPATH@/components/servicesComponents.manifest
481+
@BINPATH@/components/cryptoComponents.manifest
480482
@BINPATH@/components/TelemetryPing.js
481483
@BINPATH@/components/TelemetryPing.manifest
482484
@BINPATH@/components/messageWakeupService.js

mobile/xul/installer/package-manifest.in

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -434,6 +434,8 @@
434434
@BINPATH@/components/WeaveCrypto.manifest
435435
@BINPATH@/components/WeaveCrypto.js
436436
#endif
437+
@BINPATH@/components/servicesComponents.manifest
438+
@BINPATH@/components/cryptoComponents.manifest
437439
#ifdef MOZ_SERVICES_HEALTHREPORT
438440
@BINPATH@/components/HealthReportComponents.manifest
439441
@BINPATH@/components/HealthReportService.js

services/common/Makefile.in

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,10 @@ testing_modules := \
3030
utils.js \
3131
$(NULL)
3232

33+
EXTRA_COMPONENTS := \
34+
servicesComponents.manifest \
35+
$(NULL)
36+
3337
TEST_DIRS += tests
3438

3539
PREF_JS_EXPORTS = $(srcdir)/services-common.js
Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
# Register resource aliases
2+
resource services-common resource://gre/modules/services-common/

services/crypto/Makefile.in

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,10 @@ modules := \
1616

1717
MODULE = services-crypto
1818

19+
EXTRA_COMPONENTS := \
20+
cryptoComponents.manifest \
21+
$(NULL)
22+
1923
CRYPTO_MODULE_FILES := $(addprefix modules/,$(modules))
2024
CRYPTO_MODULE_DEST = $(FINAL_TARGET)/modules/services-crypto
2125
INSTALL_TARGETS += CRYPTO_MODULE
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
resource services-crypto resource://gre/modules/services-crypto/

services/crypto/modules/WeaveCrypto.js

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,11 @@ WeaveCrypto.prototype = {
5757
this.prefBranch = Services.prefs.getBranch("services.sync.log.");
5858
this.prefBranch.addObserver("cryptoDebug", this.observer, false);
5959
this.observer._self = this;
60-
this.debug = this.prefBranch.getBoolPref("cryptoDebug");
60+
try {
61+
this.debug = this.prefBranch.getBoolPref("cryptoDebug");
62+
} catch (x) {
63+
this.debug = false;
64+
}
6165

6266
this.initNSS();
6367
this.initAlgorithmSettings(); // Depends on NSS.

services/sync/SyncComponents.manifest

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,5 +23,3 @@ contract @mozilla.org/network/protocol/about;1?what=sync-log {d28f8a0b-95da-48f4
2323
# Register resource aliases
2424
# (Note, for tests these are also set up in addResourceAlias)
2525
resource services-sync resource://gre/modules/services-sync/
26-
resource services-common resource://gre/modules/services-common/
27-
resource services-crypto resource://gre/modules/services-crypto/

0 commit comments

Comments
 (0)