Chromium Code Reviews| Index: net/base/network_delegate.cc |
| diff --git a/net/base/network_delegate.cc b/net/base/network_delegate.cc |
| index f18a49c47ca1eefffb5840c10529bfe7900ef620..a8db6f1dc986cc1c6d68e36d462a8756a50ff443 100644 |
| --- a/net/base/network_delegate.cc |
| +++ b/net/base/network_delegate.cc |
| @@ -8,21 +8,21 @@ |
| namespace net { |
| -int NetworkDelegate::NotifyBeforeURLRequest(URLRequest* request, |
| - OldCompletionCallback* callback, |
| - GURL* new_url) { |
| +int NetworkDelegate::NotifyBeforeURLRequest( |
| + URLRequest* request, const net::CompletionCallback& callback, |
|
csilv
2011/11/30 21:56:50
same comment as header (re: already in net namespa
James Hawkins
2011/12/01 00:40:26
Done.
|
| + GURL* new_url) { |
| DCHECK(CalledOnValidThread()); |
| DCHECK(request); |
| - DCHECK(callback); |
| + DCHECK(!callback.is_null()); |
| return OnBeforeURLRequest(request, callback, new_url); |
| } |
| -int NetworkDelegate::NotifyBeforeSendHeaders(URLRequest* request, |
| - OldCompletionCallback* callback, |
| - HttpRequestHeaders* headers) { |
| +int NetworkDelegate::NotifyBeforeSendHeaders( |
| + URLRequest* request, const net::CompletionCallback& callback, |
| + HttpRequestHeaders* headers) { |
| DCHECK(CalledOnValidThread()); |
| DCHECK(headers); |
| - DCHECK(callback); |
| + DCHECK(!callback.is_null()); |
| return OnBeforeSendHeaders(request, callback, headers); |
| } |
| @@ -34,12 +34,12 @@ void NetworkDelegate::NotifySendHeaders(URLRequest* request, |
| int NetworkDelegate::NotifyHeadersReceived( |
| URLRequest* request, |
| - OldCompletionCallback* callback, |
| + const net::CompletionCallback& callback, |
| HttpResponseHeaders* original_response_headers, |
| scoped_refptr<HttpResponseHeaders>* override_response_headers) { |
| DCHECK(CalledOnValidThread()); |
| DCHECK(original_response_headers); |
| - DCHECK(callback); |
| + DCHECK(!callback.is_null()); |
| return OnHeadersReceived(request, callback, original_response_headers, |
| override_response_headers); |
| } |