Bug 982848: Don't clear syncKeyBundle in BrowserID Identity manager during failures to fetch token. r=markh

--HG--
extra : rebase_source : 42bccbef9e86cbc2f9e6113656acdddd04844f67

services/sync/modules/browserid_identity.js

@@ -135,7 +135,7 @@ this.BrowserIDManager.prototype = {
     }
 
     // If we are already happy then there is nothing more to do.
-    if (Weave.Status.login == LOGIN_SUCCEEDED) {
+    if (this._syncKeyBundle) {
       return Promise.resolve();
     }
 
@@ -513,7 +513,6 @@ this.BrowserIDManager.prototype = {
         // This will arrange for us to be in the right 'currentAuthState'
         // such that UI will show the right error.
         this._shouldHaveSyncKeyBundle = true;
-        this._syncKeyBundle = null;
         Weave.Status.login = this._authFailureReason;
         Services.obs.notifyObservers(null, "weave:service:login:error", null);
         throw err;