Separate same domain and cross domain <link rel=...> prerenders for reporting.

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 209 matching lines @@
   DCHECK(child_id_ == -1 || session_storage_namespace);
 
   std::vector<PendingPrerenderInfo> pending_prerender_list;
   pending_prerender_list.swap(pending_prerenders_);
   for (std::vector<PendingPrerenderInfo>::iterator it =
            pending_prerender_list.begin();
        it != pending_prerender_list.end();
        ++it) {
     if (it->weak_prerender_handle && it->weak_prerender_handle->IsValid()) {
       prerender_manager_->StartPendingPrerender(
-          it->weak_prerender_handle.get(), ORIGIN_LINK_REL_PRERENDER, child_id_,
+          it->weak_prerender_handle.get(),
+          ORIGIN_LINK_REL_PRERENDER_CROSSDOMAIN,

[dominich, 2012/09/13 20:03:36]

this isn't necessarily true. Pending prerenders can be same-domain.

[gavinp, 2012/09/14 02:00:42]

Good catch. Done.

+          child_id_,
           it->url, it->referrer, it->size, session_storage_namespace);
     }
   }
 }
 
 PrerenderContents::PendingPrerenderInfo::PendingPrerenderInfo(
     const base::WeakPtr<PrerenderHandle> weak_prerender_handle,
     const GURL& url,
     const content::Referrer& referrer,
     const gfx::Size& size)
@@ skipping 477 matching lines @@
 bool PrerenderContents::IsCrossSiteNavigationPending() const {
   if (!prerender_contents_.get() || !prerender_contents_->web_contents())
     return false;
   const WebContents* web_contents = prerender_contents_->web_contents();
   return (web_contents->GetSiteInstance() !=
           web_contents->GetPendingSiteInstance());
 }
 
 
 }  // namespace prerender