Index: chrome/test/data/extensions/api_test/webrequest/test_blocking.js |
diff --git a/chrome/test/data/extensions/api_test/webrequest/test_blocking.js b/chrome/test/data/extensions/api_test/webrequest/test_blocking.js |
index a0e6d6f8d36222de5b471948d618f50a31982850..6829044cf885a487c32c5f07834899070f8223e5 100644 |
--- a/chrome/test/data/extensions/api_test/webrequest/test_blocking.js |
+++ b/chrome/test/data/extensions/api_test/webrequest/test_blocking.js |
@@ -126,9 +126,6 @@ runTests([ |
// 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 |
@@ -140,14 +137,15 @@ runTests([ |
}, |
retval: {redirectUrl: getURL("simpleLoad/a.html")} |
}, |
- { label: "onErrorOccurred-1", |
- event: "onErrorOccurred", |
+ { label: "onBeforeRedirect", |
+ event: "onBeforeRedirect", |
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", |
event: "onBeforeRequest", |
@@ -178,7 +176,7 @@ runTests([ |
}, |
], |
[ // event order |
- ["onBeforeRequest-1", "onErrorOccurred-1", "onBeforeRequest-2", |
+ ["onBeforeRequest-1", "onBeforeRedirect", "onBeforeRequest-2", |
"onResponseStarted", "onCompleted"], |
], |
{}, // filter |