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

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

Issue 8889007: Enable TransferNavigationResourceHandler (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 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
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..c1c922a9635debbeddffbf9f46b0ee6a0c664615 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,13 +137,14 @@ 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",
@@ -178,7 +176,7 @@ runTests([
},
],
[ // event order
- ["onBeforeRequest-1", "onErrorOccurred-1", "onBeforeRequest-2",
+ ["onBeforeRequest-1", "onBeforeRedirect", "onBeforeRequest-2",
"onResponseStarted", "onCompleted"],
],
{}, // filter
@@ -459,7 +457,6 @@ runTests([
responseHeadersExist: true,
},
retval_function: function(name, details) {
- console.log(JSON.stringify(details));
responseHeaders = details.responseHeaders;
var found = false;
var expectedValue = [
« no previous file with comments | « chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.cc ('k') | content/renderer/render_view_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698