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

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: Merge to p388233. 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
« no previous file with comments | « net/url_request/sdch_dictionary_fetcher_unittest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/url_request/url_request.h
diff --git a/net/url_request/url_request.h b/net/url_request/url_request.h
index 745160ea91e2ec52b6e3dd2bbf4f3dd182ba2e7d..b470d85c34f4e6da1d22b6cb7093e2daae2ae453 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
« no previous file with comments | « net/url_request/sdch_dictionary_fetcher_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698