Fix bug in HttpURLFetcher that dropped the response headers

headless/public/util/http_url_fetcher.cc

 // Copyright 2016 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 "headless/public/util/http_url_fetcher.h"
 
 #include "net/base/io_buffer.h"
 #include "net/cert/cert_status_flags.h"
+#include "net/http/http_response_headers.h"
 #include "net/url_request/url_request.h"
 #include "net/url_request/url_request_context.h"
 
 namespace headless {
 
 class HttpURLFetcher::Delegate : public net::URLRequest::Delegate {
  public:
   Delegate(const GURL& rewritten_url,
            const std::string& method,
            const net::HttpRequestHeaders& request_headers,
@@ skipping 139 matching lines @@
   if (result_code_ != net::OK) {
     result_listener_->OnFetchStartError(static_cast<net::Error>(result_code_));
     return;
   }
 
   if (net_error != net::OK) {
     result_listener_->OnFetchStartError(static_cast<net::Error>(net_error));
     return;
   }
 
-  result_listener_->OnFetchCompleteExtractHeaders(
-      request->url(), request->GetResponseCode(), bytes_read_so_far_.c_str(),
-      bytes_read_so_far_.size());
+  // TODO(alexclarke) apart from the headers there's a lot of stuff in
+  // |request->response_info()| that we drop here.  Find a way to pipe it
+  // through.
+  result_listener_->OnFetchComplete(
+      request->url(), request->GetResponseCode(),
+      request->response_info().headers,
+      bytes_read_so_far_.c_str(), bytes_read_so_far_.size());
 }
 
 HttpURLFetcher::HttpURLFetcher(
     const net::URLRequestContext* url_request_context)
     : url_request_context_(url_request_context) {}
 
 HttpURLFetcher::~HttpURLFetcher() {}
 
 void HttpURLFetcher::StartFetch(const GURL& rewritten_url,
                                 const std::string& method,
                                 const net::HttpRequestHeaders& request_headers,
                                 ResultListener* result_listener) {
   delegate_.reset(new Delegate(rewritten_url, method, request_headers,
                                url_request_context_, result_listener));
 }
 
 }  // namespace headless