From 128738870e239866f5c6a1b55b9170dbae9ae854 Mon Sep 17 00:00:00 2001
From: Richard Newman <rnewman@mozilla.com>
Date: Thu, 22 Sep 2011 13:46:28 -0700
Subject: [PATCH] Bug 672878 - follow-up to un-nest an if. a=dumb

---
 services/sync/modules/rest.js | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/services/sync/modules/rest.js b/services/sync/modules/rest.js
index 0f2d149dacc..9b62f97b2d1 100644
--- a/services/sync/modules/rest.js
+++ b/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);