Removes get upload progress plumbing. URLRequest queries the UploadDataStream directly.

net/url_request/url_request.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.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 344 matching lines @@
   if (!job_.get()) {
     // We haven't started or the request was cancelled
     return UploadProgress();
   }
   if (final_upload_progress_.position()) {
     // The first job completed and none of the subsequent series of
     // GETs when following redirects will upload anything, so we return the
     // cached results from the initial job, the POST.
     return final_upload_progress_;
   }
-  return job_->GetUploadProgress();
+  if (upload_data_stream_) {
+    return upload_data_stream_->GetUploadProgress();
+  }

[mmenke, 2016/09/12 19:35:31]

nit:  Don't use braces on single-line if.

[shivanisha, 2016/09/14 20:40:25]

done.

+  return UploadProgress();
 }
 
 void URLRequest::GetResponseHeaderByName(const string& name,
                                          string* value) const {
   DCHECK(value);
   if (response_info_.headers.get()) {
     response_info_.headers->GetNormalizedHeader(name, value);
   } else {
     value->clear();
   }
@@ skipping 578 matching lines @@
   }
 
   if (!redirect_info.new_url.is_valid())
     return ERR_INVALID_URL;
 
   if (!job_->IsSafeRedirect(redirect_info.new_url)) {
     DVLOG(1) << "disallowing redirect: unsafe protocol";
     return ERR_UNSAFE_REDIRECT;
   }
 
-  if (!final_upload_progress_.position())
-    final_upload_progress_ = job_->GetUploadProgress();
+  if (!final_upload_progress_.position() && upload_data_stream_)
+    final_upload_progress_ = upload_data_stream_->GetUploadProgress();
   PrepareToRestart();
 
   if (redirect_info.new_method != method_) {
     // TODO(davidben): This logic still needs to be replicated at the consumers.
     if (method_ == "POST") {
       // If being switched from POST, must remove Origin header.
       // TODO(jww): This is Origin header removal is probably layering violation
       // and
       // should be refactored into //content. See https://crbug.com/471397.
       extra_request_headers_.RemoveHeader(HttpRequestHeaders::kOrigin);
@@ skipping 259 matching lines @@
     out->clear();
 }
 
 void URLRequest::set_status(URLRequestStatus status) {
   DCHECK(status_.is_io_pending() || status_.is_success() ||
          (!status.is_success() && !status.is_io_pending()));
   status_ = status;
 }
 
 }  // namespace net