Added HTTP response code field to ResourceRequestDetails.

content/public/browser/resource_request_details.h

 // 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.
 
 #ifndef CONTENT_PUBLIC_BROWSER_RESOURCE_REQUEST_DETAILS_H_
 #define CONTENT_PUBLIC_BROWSER_RESOURCE_REQUEST_DETAILS_H_
 
 #include <string>
 
 #include "net/base/host_port_pair.h"
@@ skipping 22 matching lines @@
   std::string referrer;
   bool has_upload;
   int load_flags;
   int origin_child_id;
   net::URLRequestStatus status;
   int ssl_cert_id;
   net::CertStatus ssl_cert_status;
   ResourceType::Type resource_type;
   net::HostPortPair socket_address;
   int render_frame_id;
+  // HTTP response code.
+  // Defaults to 200 if there is no response code among headers.
+  // -1 if there are no response headers yet for some reason.

[Charlie Reis, 2014/04/08 23:22:29]

nit: Remove "for some reason." 

+  // Can be also 0 if the response code text seems to exist but could not be

[Charlie Reis, 2014/04/08 23:22:29]

Most of this comment seems to be taken from http_response_headers.h.  Rather
than copying and reordering it here, let's just say "See
HttpResponseHeaders::response_code()," plus your line about -1 (since that's new
behavior here).

+  // parsed.
+  int http_response_code;
 };
 
 // Details about a redirection of a resource request.
 struct ResourceRedirectDetails : public ResourceRequestDetails {
   ResourceRedirectDetails(const net::URLRequest* request,
                           int cert_id,
                           const GURL& new_url);
   virtual ~ResourceRedirectDetails();
 
   // The URL to which we are being redirected.
   GURL new_url;
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PUBLIC_BROWSER_RESOURCE_REQUEST_DETAILS_H_