XHR: avoid a redundant final progress event

See whatwg/xhr#318. This fixes a broken web platform test which happens to pass in all browsers, to also happen to pass in jsdom: web-platform-tests/wpt#27917.

Author: domenic

lib/jsdom/living/xhr/XMLHttpRequest-impl.js

@@ -884,7 +884,10 @@ function receiveResponse(xhr, response) {
     properties.timeoutFn = null;
     properties.timeoutStart = 0;
     properties.client = null;
-    fireAnEvent("progress", xhr, ProgressEvent, progressObj);
+    if (lastProgressReported !== progressObj.loaded) {
+      // https://github.com/whatwg/xhr/issues/318
+      fireAnEvent("progress", xhr, ProgressEvent, progressObj);
+    }
     readyStateChange(xhr, READY_STATES.DONE);
     fireAnEvent("load", xhr, ProgressEvent, progressObj);
     fireAnEvent("loadend", xhr, ProgressEvent, progressObj);

test/web-platform-tests/to-run.yaml

@@ -1180,6 +1180,7 @@ send-entity-body-empty.htm: [fail, hard to get Node to not send Content-Length]
 send-entity-body-get-head-async.htm: [fail, hard to get Node to not send Content-Length]
 send-entity-body-get-head.htm: [fail, hard to get Node to not send Content-Length]
 send-entity-body-none.htm: [fail-slow, hard to get Node to not send Content-Length]
+send-no-response-event-order.htm: [fail, https://github.com/whatwg/xhr/issues/318#issuecomment-792106102]
 send-redirect-post-upload.htm: [fail, Unknown]
 send-redirect-to-cors.htm: [fail, request module remove content-type header on redirect]
 send-redirect.htm: [fail, request module remove content-type header on redirect]