Introduce a delegate to avoid hardcoding "chrome-extension" in net/.

net/url_request/url_request_throttler_entry_interface.h

Index: net/url_request/url_request_throttler_entry_interface.h
diff --git a/net/url_request/url_request_throttler_entry_interface.h b/net/url_request/url_request_throttler_entry_interface.h
index e46cd2aded2a4fa5c25d72b6a07ca58967eb4d2c..7fde3ca3a3139f544b79e88352215d7cadb148d4 100644
--- a/net/url_request/url_request_throttler_entry_interface.h
+++ b/net/url_request/url_request_throttler_entry_interface.h
@@ -15,6 +15,7 @@
 
 namespace net {
 
+class URLRequest;
 class URLRequestThrottlerHeaderInterface;
 
 // Interface provided on entries of the URL request throttler manager.
@@ -26,11 +27,13 @@ class NET_EXPORT URLRequestThrottlerEntryInterface
   // Returns true when we have encountered server errors and are doing
   // exponential back-off, unless the request has |load_flags| (from
   // net/base/load_flags.h) that mean it is likely to be
-  // user-initiated.
+  // user-initiated, or the URLRequestThrottlerManagerDelegate returns
+  // false for |MayRejectRequest(request)|.
   //
   // URLRequestHttpJob checks this method prior to every request; it
   // cancels requests if this method returns true.
-  virtual bool ShouldRejectRequest(int load_flags) const = 0;
+  virtual bool ShouldRejectRequest(URLRequest* request,

[eroman, 2012/06/05 05:30:25]

Can this be |const URLRequest*| or |const URLRequest&| ?

Mutable requests concern me, since if the consumer has the ability to do things
like cancel them, the callsite which exposes this has to be prepared to check
for changes.

[Jói, 2012/06/05 16:39:14]

Done.

+                                   int load_flags) const = 0;
 
   // Calculates a recommended sending time for the next request and reserves it.
   // The sending time is not earlier than the current exponential back-off