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

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

Issue 2574063003: Move ResourceHandler deferred actions ahead of external protocol handling. (Closed)
Patch Set: Reverts URLRequest changes; redirect bugfix; improved unit tests. Created 4 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
« no previous file with comments | « no previous file | content/browser/loader/resource_loader.cc » ('j') | content/browser/loader/resource_loader.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/browser/loader/resource_loader.h
diff --git a/content/browser/loader/resource_loader.h b/content/browser/loader/resource_loader.h
index 43b7530f315444b50964f6d2ce35c76a64527ab3..1721c67f9d0af5762ad5721c15f97297dde71c0e 100644
--- a/content/browser/loader/resource_loader.h
+++ b/content/browser/loader/resource_loader.h
@@ -83,6 +83,7 @@ class CONTENT_EXPORT ResourceLoader : public net::URLRequest::Delegate,
void StartRequestInternal();
void CancelRequestInternal(int error, bool from_renderer);
+ void FollowDeferredRedirectInternal();
void CompleteResponseStarted();
void ReadMore(bool is_continuation);
void ResumeReading();
@@ -139,6 +140,9 @@ class CONTENT_EXPORT ResourceLoader : public net::URLRequest::Delegate,
bool started_request_;
int times_cancelled_after_request_start_;
+ // Stores the URL from a deferred redirect.
+ GURL deferred_redirect_url_;
mmenke 2016/12/16 16:19:36 need to include url.h
carlosk 2016/12/20 19:23:31 Done.
+
base::WeakPtrFactory<ResourceLoader> weak_ptr_factory_;
DISALLOW_COPY_AND_ASSIGN(ResourceLoader);
« no previous file with comments | « no previous file | content/browser/loader/resource_loader.cc » ('j') | content/browser/loader/resource_loader.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698