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

Unified Diff: chrome/test/data/extensions/api_test/streams_private/handle_mime_type/background.js

Issue 2940553003: Failure in reading Blob URL should results in network error
Patch Set: . Created 3 years, 6 months 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
Index: chrome/test/data/extensions/api_test/streams_private/handle_mime_type/background.js
diff --git a/chrome/test/data/extensions/api_test/streams_private/handle_mime_type/background.js b/chrome/test/data/extensions/api_test/streams_private/handle_mime_type/background.js
index 347a15006955c6b7e2a3563f465843a9768e83eb..29d7ce6831673f13cd313889d2e7e95129db527a 100644
--- a/chrome/test/data/extensions/api_test/streams_private/handle_mime_type/background.js
+++ b/chrome/test/data/extensions/api_test/streams_private/handle_mime_type/background.js
@@ -75,14 +75,16 @@ chrome.streamsPrivate.onExecuteMimeTypeHandler.addListener(
chrome.streamsPrivate.abort(params.streamUrl, function() {
var xhr = new XMLHttpRequest();
xhr.open("GET", params.streamUrl, false);
- xhr.send(null);
- if (xhr.status == 404) {
- chrome.test.notifyPass();
- } else {
+ xhr.onload = function() {
chrome.test.notifyFail(
- 'Expected a stream URL response of 404, got ' + xhr.status + '.');
+ 'Expected a network error, got ' + xhr.status + '.');
hasFailed = true;
}
+ try {
+ xhr.send(null);
+ } catch (e) {
+ chrome.test.notifyPass();
+ }
});
}
return;

Powered by Google App Engine
This is Rietveld 408576698