Make URLRequestJob::SetStatus private.

net/url_request/url_request_ftp_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_ftp_job.h"
 
 #include "base/compiler_specific.h"
 #include "base/location.h"
 #include "base/single_thread_task_runner.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 142 matching lines @@
     OnStartCompletedAsync(ERR_NO_SUPPORTED_PROXIES);
 }
 
 void URLRequestFtpJob::StartFtpTransaction() {
   // Create a transaction.
   DCHECK(!ftp_transaction_);
 
   ftp_request_info_.url = request_->url();
   ftp_transaction_ = ftp_transaction_factory_->CreateTransaction();
 
-  // No matter what, we want to report our status as IO pending since we will
-  // be notifying our consumer asynchronously via OnStartCompleted.
-  SetStatus(URLRequestStatus(URLRequestStatus::IO_PENDING, 0));
   int rv;
   if (ftp_transaction_) {
     rv = ftp_transaction_->Start(
         &ftp_request_info_,
         base::Bind(&URLRequestFtpJob::OnStartCompleted,
                    base::Unretained(this)),
         request_->net_log());
     if (rv == ERR_IO_PENDING)
       return;
   } else {
@@ skipping 28 matching lines @@
         request_->net_log());
     if (rv == ERR_IO_PENDING)
       return;
   }
   // The transaction started synchronously, but we need to notify the
   // URLRequest delegate via the message loop.
   OnStartCompletedAsync(rv);
 }
 
 void URLRequestFtpJob::OnStartCompleted(int result) {
-  // Clear the IO_PENDING status
-  SetStatus(URLRequestStatus());
-
   // Note that ftp_transaction_ may be NULL due to a creation failure.
   if (ftp_transaction_) {
     // FTP obviously doesn't have HTTP Content-Length header. We have to pass
     // the content size information manually.
     set_expected_content_size(
         ftp_transaction_->GetResponseInfo()->expected_content_size);
   }
 
   if (result == OK) {
     if (http_transaction_) {
@@ skipping 23 matching lines @@
 }
 
 void URLRequestFtpJob::OnReadCompleted(int result) {
   read_in_progress_ = false;
   ReadRawDataComplete(result);
 }
 
 void URLRequestFtpJob::RestartTransactionWithAuth() {
   DCHECK(auth_data_.get() && auth_data_->state == AUTH_STATE_HAVE_AUTH);
 
-  // No matter what, we want to report our status as IO pending since we will
-  // be notifying our consumer asynchronously via OnStartCompleted.
-  SetStatus(URLRequestStatus(URLRequestStatus::IO_PENDING, 0));
-
   int rv;
   if (proxy_info_.is_direct()) {
     rv = ftp_transaction_->RestartWithAuth(
         auth_data_->credentials,
         base::Bind(&URLRequestFtpJob::OnStartCompleted,
                    base::Unretained(this)));
   } else {
     rv = http_transaction_->RestartWithAuth(
         auth_data_->credentials,
         base::Bind(&URLRequestFtpJob::OnStartCompleted,
@@ skipping 113 matching lines @@
   if (cached_auth) {
     // Retry using cached auth data.
     SetAuth(cached_auth->credentials);
   } else {
     // Prompt for a username/password.
     NotifyHeadersComplete();
   }
 }
 
 }  // namespace net