Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(81)

Unified Diff: third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/cross-origin-preflight-get-response-type-blob.html

Issue 2550133002: Make cross-origin-preflight-get-response-type-blob fail when loading fails (Closed)
Patch Set: fix Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/cross-origin-preflight-get-response-type-blob.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/cross-origin-preflight-get-response-type-blob.html b/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/cross-origin-preflight-get-response-type-blob.html
index 191946944ebf4b054c7b747e7911251e1d889350..037ffbad30f9106a106e36a02a69ad7e3dfd4750 100644
--- a/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/cross-origin-preflight-get-response-type-blob.html
+++ b/third_party/WebKit/LayoutTests/http/tests/xmlhttprequest/cross-origin-preflight-get-response-type-blob.html
@@ -21,17 +21,23 @@ function test()
xhr.open("GET", "http://localhost:8000/xmlhttprequest/resources/cross-origin-preflight-get.php", true);
// Make this a "non-simple" cross-origin request by adding a custom header.
xhr.setRequestHeader("X-Proprietary-Header", "foo");
- xhr.onerror = function() { log("onerror") }
+ xhr.onerror = function() {
+ log("xhr onerror");
+ if (window.testRunner)
+ testRunner.notifyDone();
+ }
xhr.onload = function() {
var reader = new FileReader;
reader.onload = function(evt) {
log(evt.target.result);
+ };
+ reader.onerror = () => {
+ log("reader.onerror");
+ };
+ reader.onloadend = () => {
if (window.testRunner)
testRunner.notifyDone();
};
- reader.onerror = function () {
- log("reader.onerror");
- }
reader.readAsText(xhr.response);
}
xhr.send(null);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698