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

chrome/browser/prerender/prerender_contents.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 "chrome/browser/prerender/prerender_contents.h"
 
 #include <algorithm>
 #include <functional>
 #include <utility>
 
@@ skipping 308 matching lines @@
       this, content::NOTIFICATION_RESOURCE_RECEIVED_REDIRECT,
       content::Source<WebContents>(prerender_contents_.get()));
 
   // Transfer over the user agent override.
   prerender_contents_.get()->SetUserAgentOverride(
       prerender_manager_->config().user_agent_override);
 
   content::NavigationController::LoadURLParams load_url_params(
       prerender_url_);
   load_url_params.referrer = referrer_;
-  load_url_params.transition_type = (origin_ == ORIGIN_OMNIBOX ?
-      content::PAGE_TRANSITION_TYPED : content::PAGE_TRANSITION_LINK);
+  load_url_params.transition_type = content::PageTransitionFromInt(
+      (origin_ == ORIGIN_OMNIBOX ?
+          content::PAGE_TRANSITION_TYPED : content::PAGE_TRANSITION_LINK) |
+      content::PAGE_TRANSITION_PRERENDER);

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

nit: please use a temp variable for this argument to make this readable.  There
is some justification for using the ?: to assign an initial value... but by the
time you pass it into a function... this gets to be far less readable.

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

Done.

   load_url_params.override_user_agent =
       prerender_manager_->config().is_overriding_user_agent ?
       content::NavigationController::UA_OVERRIDE_TRUE :
       content::NavigationController::UA_OVERRIDE_FALSE;
   prerender_contents_.get()->GetController().LoadURLWithParams(load_url_params);
 }
 
 bool PrerenderContents::GetChildId(int* child_id) const {
   CHECK(child_id);
   DCHECK_GE(child_id_, -1);
@@ skipping 368 matching lines @@
 
 bool PrerenderContents::IsCrossSiteNavigationPending() const {
   if (!prerender_contents_)
     return false;
   return (prerender_contents_->GetSiteInstance() !=
           prerender_contents_->GetPendingSiteInstance());
 }
 
 
 }  // namespace prerender