Add metrics for calculating precision/recall of link navigation pre-connect triggers.

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 7b92c845689f6e216a0b357ff5db441f970a282a..ec275f705858ed9918c4ea790ee1a379eb67b634 100644
--- a/content/public/browser/resource_request_details.cc
+++ b/content/public/browser/resource_request_details.cc
@@ -11,7 +11,8 @@
 namespace content {
 
 ResourceRequestDetails::ResourceRequestDetails(const net::URLRequest* request,

[jar (doing other things), 2013/06/18 01:33:49]

Question: Why isn't this a const ref? generally we pass either pointers, or
const refs.... and the latter makes sense here.

[kouhei (in TOK), 2013/06/18 22:28:41]

I'd like to address this in separate CL.

-                                               int cert_id)
+                                               int cert_id,
+                                               PageTransition transition)
     : url(request->url()),
       original_url(request->original_url()),
       method(request->method()),
@@ -21,7 +22,8 @@ ResourceRequestDetails::ResourceRequestDetails(const net::URLRequest* request,
       status(request->status()),
       ssl_cert_id(cert_id),
       ssl_cert_status(request->ssl_info().cert_status),
-      socket_address(request->GetSocketAddress()) {
+      socket_address(request->GetSocketAddress()),
+      page_transition(transition) {
   const ResourceRequestInfo* info = ResourceRequestInfo::ForRequest(request);
   resource_type = info->GetResourceType();
   frame_id = info->GetFrameID();
@@ -44,8 +46,9 @@ ResourceRequestDetails::~ResourceRequestDetails() {}
 
 ResourceRedirectDetails::ResourceRedirectDetails(const net::URLRequest* request,
                                                  int cert_id,
+                                                 PageTransition transition,
                                                  const GURL& new_url)
-    : ResourceRequestDetails(request, cert_id),
+    : ResourceRequestDetails(request, cert_id, transition),
       new_url(new_url) {
 }