Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Replace XMLHttpRequest usage with the Fetch API in send (in test/unit/testreporter.js)

Browse Source 
Besides converting the `send` function to use the Fetch API, this patch also changes the method to return a `Promise` to get rid of the callback function. (Although, currently there's no call-site passing in a callback function.)
This commit is contained in:
Jonas Jenwald 2022-03-10 12:55:08 +01:00
parent ee39499a5a
commit e08e3f4d37
1 changed files with 24 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

42
test/unit/testreporter.js
View File

@ -1,23 +1,29 @@
const TestReporter = function (browser) {
function send(action, json, cb) {
const r = new XMLHttpRequest();
// (The POST URI is ignored atm.)
r.open("POST", action, true);
r.setRequestHeader("Content-Type", "application/json");
r.onreadystatechange = function sendTaskResultOnreadystatechange(e) {
if (r.readyState === 4) {
// Retry until successful
if (r.status !== 200) {
send(action, json, cb);
} else {
if (cb) {
cb();
function send(action, json) {
return new Promise(resolve => {
json.browser = browser;
fetch(action, {
method: "POST",
headers: {
"Content-Type": "application/json",
},
body: JSON.stringify(json),
})
.then(response => {
// Retry until successful.
if (!response.ok || response.status !== 200) {
throw new Error(response.statusText);
}
}
}
};
json.browser = browser;
r.send(JSON.stringify(json));
resolve();
})
.catch(reason => {
console.warn(`TestReporter - send failed (${action}): ${reason}`);
resolve();
send(action, json);
});
});
}
function sendInfo(message) {