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 319 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);
+  content::PageTransition transition_type_no_qualifier =
+      origin_ == ORIGIN_OMNIBOX ?
+          content::PAGE_TRANSITION_TYPED : content::PAGE_TRANSITION_LINK;

[jar (doing other things), 2013/08/01 22:56:47]

nit: The old format was bad... your format is better... more readable may be:

content::PageTransition transition_type_no_qualifier =
    origin_ == ORIGIN_OMNIBOX ? content::PAGE_TRANSITION_TYPED
                              : content::PAGE_TRANSITION_LINK;

[kouhei (in TOK), 2013/08/02 06:18:03]

Done.

+  load_url_params.transition_type = content::PageTransitionFromInt(
+      transition_type_no_qualifier | content::PAGE_TRANSITION_PRERENDER);
   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