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

Unified Diff: content/common/net/url_fetcher_core.h

Issue 10203002: Make URLRequestThrottlerManager a member of URLRequestContext. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Respond to review comments. Created 8 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: content/common/net/url_fetcher_core.h
diff --git a/content/common/net/url_fetcher_core.h b/content/common/net/url_fetcher_core.h
index db190a45fb1978e0cc51441091ef464657319dea..387c75cac446f5dd2881dff69ca1ed907af7bf4c 100644
--- a/content/common/net/url_fetcher_core.h
+++ b/content/common/net/url_fetcher_core.h
@@ -338,13 +338,19 @@ class URLFetcherCore
// Used to determine how long to wait before making a request or doing a
// retry.
+ //
// 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.
+ //
+ // 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.
+ //
+ // Both of these will be NULL if
+ // URLRequestContext::throttler_manager() is NULL.
scoped_refptr<net::URLRequestThrottlerEntryInterface>
original_url_throttler_entry_;
scoped_refptr<net::URLRequestThrottlerEntryInterface> url_throttler_entry_;
« no previous file with comments | « chrome/service/cloud_print/cloud_print_url_fetcher_unittest.cc ('k') | content/common/net/url_fetcher_core.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698