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

Unified Diff: content/browser/loader/cross_site_resource_handler.h

Issue 1384113002: CrossSiteResourceHandler: cancel request if the RFH is gone (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@no_isolate_apps
Patch Set: Add EXPECT_TRUE's Created 5 years, 2 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: content/browser/loader/cross_site_resource_handler.h
diff --git a/content/browser/loader/cross_site_resource_handler.h b/content/browser/loader/cross_site_resource_handler.h
index b73a5a1475b799e0493ea87bd0311e2ff7638f55..db1dc84e5f77da2888914c3bc1c85e37e79c7623 100644
--- a/content/browser/loader/cross_site_resource_handler.h
+++ b/content/browser/loader/cross_site_resource_handler.h
@@ -26,6 +26,12 @@ struct TransitionLayerData;
// and not a download.
class CrossSiteResourceHandler : public LayeredResourceHandler {
public:
+ enum class NavigationDecision {
+ TRANSFER_REQUIRED,
+ USE_EXISTING_RENDERER,
+ CANCEL_REQUEST
+ };
+
CrossSiteResourceHandler(scoped_ptr<ResourceHandler> next_handler,
net::URLRequest* request);
~CrossSiteResourceHandler() override;
@@ -63,7 +69,7 @@ class CrossSiteResourceHandler : public LayeredResourceHandler {
bool OnNormalResponseStarted(ResourceResponse* response,
bool* defer);
- void ResumeOrTransfer(bool is_transfer);
+ void ResumeOrTransfer(NavigationDecision decision);
void ResumeIfDeferred();
// Called when about to defer a request. Sets |did_defer_| and logs the
« no previous file with comments | « content/browser/cross_site_transfer_browsertest.cc ('k') | content/browser/loader/cross_site_resource_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698