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

Unified Diff: chrome/test/data/extensions/api_test/webrequest/framework.js

Issue 348253002: Add CORS headers to URLRequestRedirectJob. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: use a more efficient string concatenation method Created 6 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/webrequest/framework.js
diff --git a/chrome/test/data/extensions/api_test/webrequest/framework.js b/chrome/test/data/extensions/api_test/webrequest/framework.js
index 23b1303bd038b5bf3517662b914dad1cd9ec8695..51164a35834455e4d995fc704367b32edf8085b8 100644
--- a/chrome/test/data/extensions/api_test/webrequest/framework.js
+++ b/chrome/test/data/extensions/api_test/webrequest/framework.js
@@ -203,6 +203,15 @@ function captureEvent(name, details, callback) {
}
details.frameUrl = tabAndFrameUrls[key] || "unknown frame URL";
}
+ if (name == "onBeforeRedirect") {
+ // This value is just an implementation detail of the network layer.
+ // This status code means that the redirect originates from the network
+ // layer, e.g. caused by a Chrome extension.
+ // The status line is not guaranteed to be stable.
+ if (details.statusLine == "HTTP/1.1 307 Delegate") {
+ details.statusLine = "";
+ }
+ }
// This assigns unique IDs to frames. The new IDs are only deterministic, if
// the frames documents are loaded in order. Don't write browser tests with

Powered by Google App Engine
This is Rietveld 408576698