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

content/public/browser/resource_request_details.cc

 // 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.
 
 #include "content/public/browser/resource_request_details.h"
 
 #include "content/browser/worker_host/worker_service_impl.h"
 #include "content/public/browser/resource_request_info.h"
 #include "net/url_request/url_request.h"
 
 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()),
       referrer(request->referrer()),
       has_upload(request->has_upload()),
       load_flags(request->load_flags()),
       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();
 
   // If request is from the worker process on behalf of a renderer, use
   // the renderer process id, since it consumes the notification response
   // such as ssl state etc.
   // TODO(atwilson): need to notify all associated renderers in the case
   // of ssl state change (http://crbug.com/25357). For now, just notify
   // the first one (works for dedicated workers and shared workers with
   // a single process).
   if (!WorkerServiceImpl::GetInstance()->GetRendererForWorker(
           info->GetChildID(), &origin_child_id, &origin_route_id)) {
     origin_child_id = info->GetChildID();
     origin_route_id = info->GetRouteID();
   }
 }
 
 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) {
 }
 
 ResourceRedirectDetails::~ResourceRedirectDetails() {}
 
 }  // namespace content