Index: chrome/test/data/extensions/api_test/webrequest/test_blocking.js |
=================================================================== |
--- chrome/test/data/extensions/api_test/webrequest/test_blocking.js (revision 113792) |
+++ chrome/test/data/extensions/api_test/webrequest/test_blocking.js (working copy) |
@@ -126,6 +126,9 @@ |
// Navigates to a page with a blocking handler that redirects to a different |
// page. |
+ // TODO(mpcomplete): We should see an onBeforeRedirect as well, but our |
+ // process switching logic cancels the original redirect request and |
+ // starts a new one instead. See http://crbug.com/79520. |
function complexLoadRedirected() { |
expect( |
[ // events |
@@ -137,14 +140,13 @@ |
}, |
retval: {redirectUrl: getURL("simpleLoad/a.html")} |
}, |
- { label: "onBeforeRedirect", |
- event: "onBeforeRedirect", |
+ { label: "onErrorOccurred-1", |
+ event: "onErrorOccurred", |
details: { |
url: getURL("complexLoad/a.html"), |
- redirectUrl: getURL("simpleLoad/a.html"), |
- statusLine: "", |
- statusCode: -1, |
fromCache: false, |
+ error: "net::ERR_ABORTED" |
+ // Request to chrome-extension:// url has no IP. |
} |
}, |
{ label: "onBeforeRequest-2", |
@@ -176,7 +178,7 @@ |
}, |
], |
[ // event order |
- ["onBeforeRequest-1", "onBeforeRedirect", "onBeforeRequest-2", |
+ ["onBeforeRequest-1", "onErrorOccurred-1", "onBeforeRequest-2", |
"onResponseStarted", "onCompleted"], |
], |
{urls: ["<all_urls>"]}, // filter |
@@ -457,6 +459,7 @@ |
responseHeadersExist: true, |
}, |
retval_function: function(name, details) { |
+ console.log(JSON.stringify(details)); |
responseHeaders = details.responseHeaders; |
var found = false; |
var expectedValue = [ |