[Pkg-javascript-commits] [sockjs-client] 174/350: better debug for jsonp sender

tonnerre at ancient-solutions.com tonnerre at ancient-solutions.com
Fri Aug 5 01:04:17 UTC 2016


This is an automated email from the git hooks/post-receive script.

tonnerre-guest pushed a commit to branch upstream
in repository sockjs-client.

commit 778ee1b9bfa99f555ebd61cf4fe09f0c665cc060
Author: Bryce Kahle <bkahle at gmail.com>
Date:   Fri Oct 17 16:53:18 2014 -0400

    better debug for jsonp sender
---
 lib/transport/sender/jsonp.js | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/lib/transport/sender/jsonp.js b/lib/transport/sender/jsonp.js
index 1aa3a1f..b4ca510 100644
--- a/lib/transport/sender/jsonp.js
+++ b/lib/transport/sender/jsonp.js
@@ -56,7 +56,7 @@ module.exports = function (url, payload, callback) {
   form.submit();
 
   var completed = function(err) {
-    debug('completed', err);
+    debug('completed', id, err);
     if (!iframe.onerror) {
       return;
     }
@@ -64,6 +64,7 @@ module.exports = function (url, payload, callback) {
     // Opera mini doesn't like if we GC iframe
     // immediately, thus this timeout.
     setTimeout(function() {
+      debug('cleaning up', id);
       iframe.parentNode.removeChild(iframe);
       iframe = null;
     }, 500);
@@ -72,18 +73,22 @@ module.exports = function (url, payload, callback) {
     // failed to submit our form.
     callback(err);
   };
-  iframe.onerror = completed;
+  iframe.onerror = function () {
+    debug('onerror', id);
+    completed();
+  };
   iframe.onload = function () {
+    debug('onload', id);
     completed();
   };
   iframe.onreadystatechange = function(e) {
-    debug('onreadystatechange', iframe.readyState, e);
+    debug('onreadystatechange', id, iframe.readyState, e);
     if (iframe.readyState === 'complete') {
       completed();
     }
   };
   return function () {
-    debug('aborted');
+    debug('aborted', id);
     completed(new Error('Aborted'));
   };
 };

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-javascript/sockjs-client.git



More information about the Pkg-javascript-commits mailing list