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

Unified Diff: net/url_request/url_fetcher_core.h

Issue 495523003: Change SDCHDictionaryFetcher to use URLRequest instead of URLFetcher. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: DoLoop implementation. Created 6 years, 3 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_fetcher_core.h
diff --git a/net/url_request/url_fetcher_core.h b/net/url_request/url_fetcher_core.h
index bddeaf4c88b88f4e3a96aa855c78ddd28bc6853b..584ce24fa26e10022b1e2371a0055f4b0273e2fb 100644
--- a/net/url_request/url_fetcher_core.h
+++ b/net/url_request/url_fetcher_core.h
@@ -251,13 +251,13 @@ class URLFetcherCore
//
// Both of them can only be accessed on the IO thread.
//
- // We need not only the throttler entry for |original_URL|, but also
- // the one for |url|. For example, consider the case that URL A
- // redirects to URL B, for which the server returns a 500
- // response. In this case, the exponential back-off release time of
- // URL A won't increase. If we retry without considering the
- // back-off constraint of URL B, we may send out too many requests
- // for URL A in a short period of time.
+ // To determine the proper backoff timing, throttler entries for
+ // both |original_URL| and |url| are needed. For example, consider
+ // the case that URL A redirects to URL B, for which the server
+ // returns a 500 response. In this case, the exponential back-off
+ // release time of URL A won't increase. If only the backoff
+ // constraints for URL A are considered, too many requests for URL A
+ // may be sent in a short period of time.
//
// Both of these will be NULL if
// URLRequestContext::throttler_manager() is NULL.

Powered by Google App Engine
This is Rietveld 408576698