Notify NetworkDelegate when bytes have been received over the network.

net/url_request/url_request_http_job.cc

 // Copyright (c) 2012 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.
 
 #include "net/url_request/url_request_http_job.h"
 
 #include "base/base_switches.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
@@ skipping 1523 matching lines @@
   done_ = true;
 
   // Notify NetworkQualityEstimator.
   if (request() && (reason == FINISHED || reason == ABORTED)) {
     NetworkQualityEstimator* network_quality_estimator =
         request()->context()->network_quality_estimator();
     if (network_quality_estimator)
       network_quality_estimator->NotifyRequestCompleted(*request());
   }
 
+  // Notify about any new network usage.
+  MaybeNotifyNetworkBytes();

[mmenke, 2015/08/19 15:48:41]

Why is this needed, when we call this in NotifyDone?  Cancellation?  I'd really
rather we not increase the coupling between this and its parent class.

[sclittle, 2015/08/20 01:10:49]

This notification is no longer needed now that
URLRequestHttpJob::GetTotalReceivedBytes() doesn't drop to 0 when the request is
cancelled.

+
   RecordPerfHistograms(reason);
   if (request_)
     request_->set_received_response_content_length(prefilter_bytes_read());
 }
 
 HttpResponseHeaders* URLRequestHttpJob::GetResponseHeaders() const {
   DCHECK(transaction_.get());
   DCHECK(transaction_->GetResponseInfo());
   return override_response_headers_.get() ?
              override_response_headers_.get() :
              transaction_->GetResponseInfo()->headers.get();
 }
 
 void URLRequestHttpJob::NotifyURLRequestDestroyed() {
   awaiting_callback_ = false;
 }
 
 }  // namespace net