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

Replace XMLHttpRequest usage with the Fetch API in Driver.run

Browse Source 
This is a first step in what'll hopefully become a series of patches to implement PR 14287 in smaller steps.
This commit is contained in:
Jonas Jenwald 2022-03-06 12:47:12 +01:00
parent 5242c38af5
commit 151b140eac
1 changed files with 21 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

44
test/driver.js
View File

@ -368,34 +368,32 @@ class Driver {
this._log(`Harness thinks this browser is ${this.browser}\n`);
this._log('Fetching manifest "' + this.manifestFile + '"... ');
const r = new XMLHttpRequest();
r.open("GET", this.manifestFile, false);
r.onreadystatechange = () => {
if (r.readyState === 4) {
this._log("done\n");
this.manifest = JSON.parse(r.responseText);
if (this.testFilter?.length || this.xfaOnly) {
this.manifest = this.manifest.filter(item => {
if (this.testFilter.includes(item.id)) {
return true;
}
if (this.xfaOnly && item.enableXfa) {
return true;
}
return false;
});
}
this.currentTask = 0;
this._nextTask();
}
};
if (this.delay > 0) {
this._log("\nDelaying for " + this.delay + " ms...\n");
}
// When gathering the stats the numbers seem to be more reliable
// if the browser is given more time to start.
setTimeout(function () {
r.send(null);
setTimeout(async () => {
const response = await fetch(this.manifestFile);
if (!response.ok) {
throw new Error(response.statusText);
}
this._log("done\n");
this.manifest = await response.json();
if (this.testFilter?.length || this.xfaOnly) {
this.manifest = this.manifest.filter(item => {
if (this.testFilter.includes(item.id)) {
return true;
}
if (this.xfaOnly && item.enableXfa) {
return true;
}
return false;
});
}
this.currentTask = 0;
this._nextTask();
}, this.delay);
}