Kill URLRequestJobTracker.

net/base/network_delegate.h

Index: net/base/network_delegate.h
diff --git a/net/base/network_delegate.h b/net/base/network_delegate.h
index 977df2fdc72857251ea88556c7c3a288978b35eb..313a90b9af1b8b65e909a8480f95923302ba9202 100644
--- a/net/base/network_delegate.h
+++ b/net/base/network_delegate.h
@@ -49,6 +49,7 @@ class NetworkDelegate : public base::NonThreadSafe {
   void NotifyBeforeRedirect(URLRequest* request,
                             const GURL& new_location);
   void NotifyResponseStarted(URLRequest* request);
+  void NotifyBytesRead(const URLRequest& request, int bytes_read);

[rvargas (doing something else), 2011/05/18 21:45:18]

It is interesting that bytes_read is always the raw_bytes coming from the
network, as opposed to the post-filtered byte count. That looks correct for the
(currently only) consumer of this data, but it is not clear that this is the
generic form, given that the overall interface of URLRequest hides this value.

I guess we need to at least document that...

I was actually thinking that it would be better to observe at the URLRequest
level instead of the URLRequestJob level, but if we want the raw bytes it has to
be done at the current level.

[willchan no longer on Chromium, 2011/05/18 23:55:55]

I've renamed to NotifyRawBytesRead().

   void NotifyCompleted(URLRequest* request);
   void NotifyURLRequestDestroyed(URLRequest* request);
   void NotifyHttpTransactionDestroyed(uint64 request_id);
@@ -89,6 +90,9 @@ class NetworkDelegate : public base::NonThreadSafe {
   // This corresponds to URLRequestDelegate::OnResponseStarted.
   virtual void OnResponseStarted(URLRequest* request) = 0;
 
+  // Called every time we read raw bytes.
+  virtual void OnBytesRead(const URLRequest& request, int bytes_read) = 0;
+
   // Indicates that the URL request has been completed or failed.
   virtual void OnCompleted(URLRequest* request) = 0;