Add request_id to HttpRequestInfo and pass it to the NetworkDelegate for events.

net/base/network_delegate.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef NET_BASE_NETWORK_DELEGATE_H_
 #define NET_BASE_NETWORK_DELEGATE_H_
 #pragma once
 
 #include "base/threading/non_thread_safe.h"
 #include "net/base/completion_callback.h"
@@ skipping 13 matching lines @@
 class HttpRequestHeaders;
 class URLRequest;
 class URLRequestJob;
 
 class NetworkDelegate : public base::NonThreadSafe {
  public:
   virtual ~NetworkDelegate() {}
 
   // Notification interface called by the network stack. Note that these
   // functions mostly forward to the private virtuals. They also add some sanity
-  // checking on parameters.
-  bool NotifyBeforeURLRequest(URLRequest* request,
+  // checking on parameters. See the corresponding virtuals for explanations of
+  // the methods and their arguments.
+  int NotifyBeforeURLRequest(URLRequest* request,
+                             CompletionCallback* callback);
+  int NotifyBeforeSendHeaders(uint64 request_id,
+                              HttpRequestHeaders* headers,
                               CompletionCallback* callback);
-  void NotifySendHttpRequest(HttpRequestHeaders* headers);
   void NotifyResponseStarted(URLRequest* request);
   void NotifyReadCompleted(URLRequest* request, int bytes_read);
+  void NotifyURLRequestDestroyed(URLRequest* request);
 
   // Returns a URLRequestJob that will be used to handle the request if
   // non-null.
   // TODO(koz): Currently this is called inside registered ProtocolFactories,
   // so that we can perform Delegate-dependent request handling from the static
   // factories, but ultimately it should be called directly from
   // URLRequestJobManager::CreateJob() as a general override mechanism.
   URLRequestJob* MaybeCreateURLRequestJob(URLRequest* request);
 
  private:
   // This is the interface for subclasses of NetworkDelegate to implement. This
   // member functions will be called by the respective public notification
   // member function, which will perform basic sanity checking.
 
-  // Called before a request is sent.
-  virtual bool OnBeforeURLRequest(URLRequest* request,
+  // Called before a request is sent. The callback can be called at any time,
+  // but will have no effect if the request has already been cancelled or
+  // deleted. Returns a net status code, generally either OK to continue with
+  // the request or ERR_IO_PENDING if the result is not ready yet.
+  virtual int OnBeforeURLRequest(URLRequest* request,
+                                 CompletionCallback* callback) = 0;
+
+  // Called right before the HTTP headers are sent. Allows the delegate to
+  // read/write |headers| before they get sent out. The callback can be called
+  // at any time, but will have no effect if the transaction handling this
+  // request has been cancelled. Returns a net status code.
+  virtual int OnBeforeSendHeaders(uint64 request_id,
+                                  HttpRequestHeaders* headers,
                                   CompletionCallback* callback) = 0;
 
-  // Called right before the HTTP headers are sent.  Allows the delegate to
-  // read/write |headers| before they get sent out.
-  virtual void OnSendHttpRequest(HttpRequestHeaders* headers) = 0;
-
   // This corresponds to URLRequestDelegate::OnResponseStarted.
   virtual void OnResponseStarted(URLRequest* request) = 0;
 
   // This corresponds to URLRequestDelegate::OnReadCompleted.
   virtual void OnReadCompleted(URLRequest* request, int bytes_read) = 0;
 
+  // Called when an URLRequest is being destroyed.

[willchan no longer on Chromium, 2011/03/26 01:46:49]

Please note that we're already in the destructor, so calling any method on
|request| is probably a bad idea.

[Matt Perry, 2011/03/28 22:51:01]

Added a comment. (calling accessors is fine, just nothing that can call a
virtual)

+  virtual void OnURLRequestDestroyed(URLRequest* request) = 0;
+
   // Called before a request is sent and before a URLRequestJob is created to
   // handle the request.
   virtual URLRequestJob* OnMaybeCreateURLRequestJob(URLRequest* request) = 0;
+
 };
 
 }  // namespace net
 
 #endif  // NET_BASE_NETWORK_DELEGATE_H_