From: Mark Donnelly <mark@painless-security.com>
Date: Thu, 2 Apr 2015 21:31:15 +0000 (-0400)
Subject: Remove a wasted check of XMLHttpResponse state
X-Git-Url: http://www.project-moonshot.org/gitweb/?p=gssweb.git;a=commitdiff_plain;h=df42385e9e7f0a716d5278edb233d733b4839a49

Remove a wasted check of XMLHttpResponse state

The state was checked immediately after the send() call - which yields unpredictable results because of the inherent multithreading - and then does nothing more than log a message about it.  So, this winds up logging error messages at times when everything is perfectly fine, but doesn't recover from any sort of error.
---

diff --git a/navigator.gssweb.js b/navigator.gssweb.js
index 8aa498c..cd17389 100644
--- a/navigator.gssweb.js
+++ b/navigator.gssweb.js
@@ -148,9 +148,6 @@ var GSSWeb = (function () {
     this.xhr.onreadystatechange = this.recvTokenFromServer.bind(this);
 
     this.xhr.send(msg);
-    if (this.xhr.readyStatus <= 2)
-        console.log("Error sending POST, readyStatus =" + this.xhr.readyStatus
-		   + ", msg = '" + msg + "'");
   };
 
   GSSWeb.prototype.recvTokenFromServer = function () {