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

Unified Diff: net/base/network_delegate.h

Issue 10440119: Introduce a delegate to avoid hardcoding "chrome-extension" in net/. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Respond to review requests. Created 8 years, 6 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/base/network_delegate.h
diff --git a/net/base/network_delegate.h b/net/base/network_delegate.h
index 2886d486d91ddf7797c914b0ea68fc1e7a27711d..03242c6e5a3583b884e9f4e96aa30967a63b4c48 100644
--- a/net/base/network_delegate.h
+++ b/net/base/network_delegate.h
@@ -85,6 +85,7 @@ class NetworkDelegate : public base::NonThreadSafe {
CookieOptions* options);
bool CanAccessFile(const URLRequest& request,
const FilePath& path) const;
+ bool CanRejectRequest(const URLRequest& request) const;
private:
// This is the interface for subclasses of NetworkDelegate to implement. These
@@ -202,6 +203,10 @@ class NetworkDelegate : public base::NonThreadSafe {
virtual bool OnCanAccessFile(const URLRequest& request,
const FilePath& path) const = 0;
+ // Returns true if the given request may be rejected when the
+ // URLRequestThrottlerManager believes the server servicing the
+ // request is overloaded or down.
+ virtual bool OnCanRejectRequest(const URLRequest& request) const = 0;
eroman 2012/06/06 04:52:11 nit: Can the name have the name "throttle" in it s
Jói 2012/06/06 13:38:55 Switched naming to CanThrottleRequest / OnCanThrot
};
} // namespace net

Powered by Google App Engine
This is Rietveld 408576698