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

content/public/browser/resource_request_details.h

Index: content/public/browser/resource_request_details.h
diff --git a/content/public/browser/resource_request_details.h b/content/public/browser/resource_request_details.h
index 4e944183b315a492141b9e7f4fb997b69fa51df7..4a303b4ac4032b15403adf2c263796d3801bd4ed 100644
--- a/content/public/browser/resource_request_details.h
+++ b/content/public/browser/resource_request_details.h
@@ -7,6 +7,7 @@
 
 #include <string>
 
+#include "content/public/common/page_transition_types.h"
 #include "googleurl/src/gurl.h"
 #include "net/cert/cert_status_flags.h"
 #include "net/base/host_port_pair.h"
@@ -23,7 +24,9 @@ namespace content {
 // resource request notification. It copies many of the publicly accessible
 // member variables of net::URLRequest, but exists on the UI thread.
 struct ResourceRequestDetails {
-  ResourceRequestDetails(const net::URLRequest* request, int cert_id);
+  ResourceRequestDetails(const net::URLRequest* request,
+                         int cert_id,
+                         PageTransition transition);
 
   virtual ~ResourceRequestDetails();
 
@@ -41,12 +44,14 @@ struct ResourceRequestDetails {
   ResourceType::Type resource_type;
   net::HostPortPair socket_address;
   int64 frame_id;
+  PageTransition page_transition;
 };
 
 // Details about a redirection of a resource request.
 struct ResourceRedirectDetails : public ResourceRequestDetails {

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

nit: These should probably be classes.  By the time you have virtual methods, a
class seems much more appropriate.  

Any justification I'm missing?

Alternatively... why do we need this as a derived class (er...um... stuct)?

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

I'd like to address this in separate CL.

   ResourceRedirectDetails(const net::URLRequest* request,
                           int cert_id,
+                          PageTransition transition,
                           const GURL& new_url);
   virtual ~ResourceRedirectDetails();