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

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

Issue 8879051: Revert 113785 - 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
===================================================================
--- 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 = [
« 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