Remove URLRequestJob::GetResponseCode implementations

content/browser/streams/stream_url_request_job.cc

 // Copyright (c) 2013 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 "content/browser/streams/stream_url_request_job.h"
 
 #include "base/location.h"
 #include "base/single_thread_task_runner.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/threading/thread_task_runner_handle.h"
@@ skipping 120 matching lines @@
 
   // TODO(zork): Support registered MIME types if needed.
   return response_info_->headers->GetMimeType(mime_type);
 }
 
 void StreamURLRequestJob::GetResponseInfo(net::HttpResponseInfo* info) {
   if (response_info_)
     *info = *response_info_;
 }
 
-int StreamURLRequestJob::GetResponseCode() const {
-  if (!response_info_)
-    return -1;
-
-  return response_info_->headers->response_code();
-}
-

[tyoshino (SeeGerritForStatus), 2017/04/18 05:06:54]

OK, but it looks URLRequestJob looks at URLRequest::response_info_ which
StreamURLRequestJob doesn't update. What's the new recommended way to update
URLRequest::response_info_ that maybe we should do in
StreamURLRequestJob::HeadersCompleted?

 int64_t StreamURLRequestJob::GetTotalReceivedBytes() const {
   int64_t total_received_bytes = 0;
   if (response_info_)
     total_received_bytes = response_info_->headers->raw_headers().size();
   if (stream_.get())
     total_received_bytes += total_bytes_read_;
   return total_received_bytes;
 }
 
 void StreamURLRequestJob::DidStart() {
@@ skipping 68 matching lines @@
 }
 
 void StreamURLRequestJob::ClearStream() {
   if (stream_.get()) {
     stream_->RemoveReadObserver(this);
     stream_ = NULL;
   }
 }
 
 }  // namespace content