perf: replace responseType default value with text

ACTCD · ACTCD · commit e653d406eb3b · 2024-09-04T21:54:32.000+08:00
diff --git a/src/ext/background/main.js b/src/ext/background/main.js
@@ -453,6 +453,8 @@ async function handleMessage(message, sender) {
 			}
 			xhr.open(method, details.url, true, user, password);
 			xhr.responseType = details.responseType;
+			// avoid unexpected behavior of legacy defaults such as parsing XML
+			if (xhr.responseType === "") xhr.responseType = "text";
 			// transfer to content script via arraybuffer and then parse to blob
 			if (xhr.responseType === "blob") xhr.responseType = "arraybuffer";
 			// transfer to content script via text and then parse to document

Original file line number	Diff line number	Diff line change
`@@ -453,6 +453,8 @@ async function handleMessage(message, sender) {`
`453`	`453`	`}`
`454`	`454`	`xhr.open(method, details.url, true, user, password);`
`455`	`455`	`xhr.responseType = details.responseType;`
	`456`	`+ // avoid unexpected behavior of legacy defaults such as parsing XML`
	`457`	`+ if (xhr.responseType === "") xhr.responseType = "text";`
`456`	`458`	`// transfer to content script via arraybuffer and then parse to blob`
`457`	`459`	`if (xhr.responseType === "blob") xhr.responseType = "arraybuffer";`
`458`	`460`	`// transfer to content script via text and then parse to document`