From f999f544aad04069b03704d994a99352263f600b Mon Sep 17 00:00:00 2001
From: wolfbeast <mcwerewolf@wolfbeast.com>
Date: Wed, 13 Mar 2019 16:36:55 +0100
Subject: Remove fxAccountsEnabled checks.

Tag #812
---
 .../basilisk/base/content/browser-syncui.js        | 24 +++-------------------
 application/basilisk/base/content/sync/utils.js    | 13 ++----------
 2 files changed, 5 insertions(+), 32 deletions(-)

(limited to 'application/basilisk/base/content')

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 () {
-- 
cgit v1.2.3