summaryrefslogtreecommitdiffstats
path: root/application/basilisk
diff options
context:
space:
mode:
authorwolfbeast <mcwerewolf@wolfbeast.com>2019-03-13 16:36:55 +0100
committerwolfbeast <mcwerewolf@wolfbeast.com>2019-03-13 16:36:55 +0100
commitf999f544aad04069b03704d994a99352263f600b (patch)
treeccb01180c10cbd67a14bccce4c3e14e2044ad463 /application/basilisk
parentaa1a39311af0516dd6d2610a89b45221b60f3f20 (diff)
downloadUXP-f999f544aad04069b03704d994a99352263f600b.tar
UXP-f999f544aad04069b03704d994a99352263f600b.tar.gz
UXP-f999f544aad04069b03704d994a99352263f600b.tar.lz
UXP-f999f544aad04069b03704d994a99352263f600b.tar.xz
UXP-f999f544aad04069b03704d994a99352263f600b.zip
Remove fxAccountsEnabled checks.
Tag #812
Diffstat (limited to 'application/basilisk')
-rw-r--r--application/basilisk/base/content/browser-syncui.js24
-rw-r--r--application/basilisk/base/content/sync/utils.js13
-rw-r--r--application/basilisk/components/preferences/in-content/sync.js32
3 files changed, 13 insertions, 56 deletions
diff --git a/application/basilisk/base/content/browser-syncui.js b/application/basilisk/base/content/browser-syncui.js
index 000c8f7b4..b796e663b 100644
--- a/application/basilisk/base/content/browser-syncui.js
+++ b/application/basilisk/base/content/browser-syncui.js
@@ -93,14 +93,6 @@ var gSyncUI = {
// Returns a promise that resolves with true if Sync needs to be configured,
// false otherwise.
_needsSetup() {
- // If Sync is configured for FxAccounts then we do that promise-dance.
- if (this.weaveService.fxAccountsEnabled) {
- return fxAccounts.getSignedInUser().then(user => {
- // We want to treat "account needs verification" as "needs setup".
- return !(user && user.verified);
- });
- }
- // We are using legacy sync - check that.
let firstSync = "";
try {
firstSync = Services.prefs.getCharPref("services.sync.firstSync");
@@ -114,19 +106,9 @@ var gSyncUI = {
// to Sync needs to be verified, false otherwise.
_needsVerification() {
// For callers who care about the distinction between "needs setup" and
- // "needs verification"
- if (this.weaveService.fxAccountsEnabled) {
- return fxAccounts.getSignedInUser().then(user => {
- // If there is no user, they can't be in a "needs verification" state.
- if (!user) {
- return false;
- }
- return !user.verified;
- });
- }
-
- // Otherwise we are configured for legacy Sync, which has no verification
- // concept.
+ // "needs verification". Only for fxAccounts. XXX: remove this check.
+ // Since we are configured for legacy Sync only, which has no verification
+ // concept, just return false.
return Promise.resolve(false);
},
diff --git a/application/basilisk/base/content/sync/utils.js b/application/basilisk/base/content/sync/utils.js
index 92981f7b4..1422c1d15 100644
--- a/application/basilisk/base/content/sync/utils.js
+++ b/application/basilisk/base/content/sync/utils.js
@@ -14,13 +14,6 @@ var gSyncUtils = {
return this.bundle = Services.strings.createBundle("chrome://browser/locale/syncSetup.properties");
},
- get fxAccountsEnabled() {
- let service = Components.classes["@mozilla.org/weave/service;1"]
- .getService(Components.interfaces.nsISupports)
- .wrappedJSObject;
- return service.fxAccountsEnabled;
- },
-
// opens in a new window if we're in a modal prefwindow world, in a new tab otherwise
_openLink: function (url) {
let thisDocEl = document.documentElement,
@@ -78,8 +71,7 @@ var gSyncUtils = {
},
get tosURL() {
- let root = this.fxAccountsEnabled ? "fxa." : "";
- return Weave.Svc.Prefs.get(root + "termsURL");
+ return Weave.Svc.Prefs.get("termsURL");
},
openToS: function () {
@@ -87,8 +79,7 @@ var gSyncUtils = {
},
get privacyPolicyURL() {
- let root = this.fxAccountsEnabled ? "fxa." : "";
- return Weave.Svc.Prefs.get(root + "privacyURL");
+ return Weave.Svc.Prefs.get("privacyURL");
},
openPrivacyPolicy: function () {
diff --git a/application/basilisk/components/preferences/in-content/sync.js b/application/basilisk/components/preferences/in-content/sync.js
index 2600677a8..32b94de86 100644
--- a/application/basilisk/components/preferences/in-content/sync.js
+++ b/application/basilisk/components/preferences/in-content/sync.js
@@ -92,19 +92,7 @@ var gSyncPane = {
} catch (e) {}
if (!username) {
this.page = FXA_PAGE_LOGGED_OUT;
- } else if (xps.fxAccountsEnabled) {
- // Use cached values while we wait for the up-to-date values
- let cachedComputerName;
- try {
- cachedComputerName = Services.prefs.getCharPref("services.sync.client.name");
- }
- catch (e) {
- cachedComputerName = "";
- }
- document.getElementById("fxaEmailAddress1").textContent = username;
- this._populateComputerName(cachedComputerName);
- this.page = FXA_PAGE_LOGGED_IN;
- } else { // Old Sync
+ } else {
this.page = PAGE_HAS_ACCOUNT;
}
},
@@ -388,17 +376,13 @@ var gSyncPane = {
.getService(Components.interfaces.nsISupports)
.wrappedJSObject;
- if (service.fxAccountsEnabled) {
- this._openAboutAccounts();
- } else {
- let win = Services.wm.getMostRecentWindow("Weave:AccountSetup");
- if (win)
- win.focus();
- else {
- window.openDialog("chrome://browser/content/sync/setup.xul",
- "weaveSetup", "centerscreen,chrome,resizable=no",
- wizardType);
- }
+ let win = Services.wm.getMostRecentWindow("Weave:AccountSetup");
+ if (win)
+ win.focus();
+ else {
+ window.openDialog("chrome://browser/content/sync/setup.xul",
+ "weaveSetup", "centerscreen,chrome,resizable=no",
+ wizardType);
}
},