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

Unified Diff: net/base/network_delegate.cc

Issue 6698009: Add request_id to HttpRequestInfo and pass it to the NetworkDelegate for events. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: net and cache intercept Created 9 years, 9 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.cc
diff --git a/net/base/network_delegate.cc b/net/base/network_delegate.cc
index 17da7a8ea04b23eab8126609e905e6b4648c5f68..a2f06f95400cbc844b29d0fbc8c9b3e7923715bd 100644
--- a/net/base/network_delegate.cc
+++ b/net/base/network_delegate.cc
@@ -16,10 +16,13 @@ bool NetworkDelegate::NotifyBeforeURLRequest(URLRequest* request,
return OnBeforeURLRequest(request, callback);
}
-void NetworkDelegate::NotifySendHttpRequest(HttpRequestHeaders* headers) {
+bool NetworkDelegate::NotifyBeforeHttpRequest(uint64 request_id,
willchan no longer on Chromium 2011/03/17 15:55:54 Should we DCHECK_NE(0, request_id)?
Matt Perry 2011/03/22 21:11:43 Not all HTTP requests come from URLRequests, so th
+ HttpRequestHeaders* headers,
+ CompletionCallback* callback) {
DCHECK(CalledOnValidThread());
DCHECK(headers);
- OnSendHttpRequest(headers);
+ DCHECK(callback);
+ return OnBeforeHttpRequest(request_id, headers, callback);
}
void NetworkDelegate::NotifyResponseStarted(URLRequest* request) {

Powered by Google App Engine
This is Rietveld 408576698