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

Unified Diff: net/url_request/url_request.h

Issue 1880283002: Disallow redirects in SDCH dictionary fetches. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 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: net/url_request/url_request.h
diff --git a/net/url_request/url_request.h b/net/url_request/url_request.h
index fcbc4454b8cf65f96858d3488e9d665a08fee529..92df4eb94332ce692558048e633ed4d40eeaf83c 100644
--- a/net/url_request/url_request.h
+++ b/net/url_request/url_request.h
@@ -149,12 +149,13 @@ class NET_EXPORT URLRequest : NON_EXPORTED_BASE(public base::NonThreadSafe),
// redirect call.
//
// When this function is called, the request will still contain the
- // original URL, the destination of the redirect is provided in 'new_url'.
- // If the delegate does not cancel the request and |*defer_redirect| is
- // false, then the redirect will be followed, and the request's URL will be
- // changed to the new URL. Otherwise if the delegate does not cancel the
- // request and |*defer_redirect| is true, then the redirect will be
- // followed once FollowDeferredRedirect is called on the URLRequest.
+ // original URL, the destination of the redirect is provided in
+ // |redirect_info.new_url|. If the delegate does not cancel the request
+ // and |*defer_redirect| is false, then the redirect will be followed, and
+ // the request's URL will be changed to the new URL. Otherwise if the
+ // delegate does not cancel the request and |*defer_redirect| is true, then
+ // the redirect will be followed once FollowDeferredRedirect is called
+ // on the URLRequest.
//
// The caller must set |*defer_redirect| to false, so that delegates do not
// need to set it if they are happy with the default behavior of not

Powered by Google App Engine
This is Rietveld 408576698