Added HTTP response code field to ResourceRequestDetails.

content/public/browser/resource_request_details.cc

Index: content/public/browser/resource_request_details.cc
diff --git a/content/public/browser/resource_request_details.cc b/content/public/browser/resource_request_details.cc
index c24adebc4aa381bd023142a60fa00ddd2b38f32a..22ed2a95369beae96bdfaed71c815669a4bb23c0 100644
--- a/content/public/browser/resource_request_details.cc
+++ b/content/public/browser/resource_request_details.cc
@@ -6,6 +6,7 @@
 
 #include "content/browser/worker_host/worker_service_impl.h"
 #include "content/public/browser/resource_request_info.h"
+#include "net/http/http_response_headers.h"
 #include "net/url_request/url_request.h"
 
 namespace content {
@@ -25,6 +26,10 @@ ResourceRequestDetails::ResourceRequestDetails(const net::URLRequest* request,
   const ResourceRequestInfo* info = ResourceRequestInfo::ForRequest(request);
   resource_type = info->GetResourceType();
   render_frame_id = info->GetRenderFrameID();
+  http_response_code =
+      request->response_info().headers ?

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

Seems like we want a .get() here to be clear.

+          request->response_info().headers.get()->response_code()
+          : -1;

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

nit: The colon belongs on the previous line.  (It's reasonable to put the -1 on
the previous line as well.)

 
   // If request is from the worker process on behalf of a renderer, use
   // the renderer process id, since it consumes the notification response