Bug 672878 - follow-up to un-nest an if. a=dumb

services/sync/modules/rest.js

@@ -425,20 +425,20 @@ RESTRequest.prototype = {
       // From nsIRequest.
       requestStatus = channel.status;
       this._log.trace("Request status is " + requestStatus);
-      if (statusSuccess && (statusCode != requestStatus)) {
-        this._log.error("Request status " + requestStatus +
-                        " does not match status arg " + statusCode);
-        try {
-          channel.responseStatus;
-        } catch (ex) {
-          this._log.error("... and we got " + Utils.exceptionStr(ex) +
-                          " retrieving responseStatus.");
-        }
-      }
     } catch (ex) {
       this._log.warn("Got exception " + Utils.exceptionStr(ex) +
                      " fetching channel.status.");
     }
+    if (statusSuccess && (statusCode != requestStatus)) {
+      this._log.error("Request status " + requestStatus +
+                      " does not match status arg " + statusCode);
+      try {
+        channel.responseStatus;
+      } catch (ex) {
+        this._log.error("... and we got " + Utils.exceptionStr(ex) +
+                        " retrieving responseStatus.");
+      }
+    }
 
     let requestStatusSuccess = Components.isSuccessCode(requestStatus);