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

chrome/browser/prerender/prerender_manager.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_manager.h"
 
 #include <algorithm>
 #include <functional>
 #include <string>
 #include <vector>
@@ skipping 209 matching lines @@
     int route_id,
     const GURL& url,
     const content::Referrer& referrer,
     const gfx::Size& size) {
 #if defined(OS_ANDROID)
   // TODO(jcivelli): http://crbug.com/113322 We should have an option to disable
   //                link-prerender and enable omnibox-prerender only.
   return NULL;
 #else
   DCHECK(!size.IsEmpty());
-  if (PrerenderData* parent_prerender_data =
-          FindPrerenderDataForChildAndRoute(process_id, route_id)) {
-    // Instead of prerendering from inside of a running prerender, we will defer
-    // this request until its launcher is made visible.
-    if (PrerenderContents* contents = parent_prerender_data->contents_) {
-      pending_prerender_list_.push_back(
-          linked_ptr<PrerenderData>(new PrerenderData(this)));
-      PrerenderHandle* prerender_handle =
-          new PrerenderHandle(pending_prerender_list_.back().get());
-      contents->AddPendingPrerender(
-          prerender_handle->weak_ptr_factory_.GetWeakPtr(),
-          ORIGIN_LINK_REL_PRERENDER, url, referrer, size);
-      return prerender_handle;
-    }
-  }
-
+  bool is_same_domain = false;
+  SessionStorageNamespace* session_storage_namespace = NULL;
   // Unit tests pass in a process_id == -1.
-  SessionStorageNamespace* session_storage_namespace = NULL;
   if (process_id != -1) {
     RenderViewHost* source_render_view_host =
         RenderViewHost::FromID(process_id, route_id);
     if (!source_render_view_host)
       return NULL;
     WebContents* source_web_contents =
         WebContents::FromRenderViewHost(source_render_view_host);
     if (!source_web_contents)
       return NULL;
+    is_same_domain = (source_web_contents->GetURL().host() == url.host());

[dominich, 2012/10/08 17:03:04]

does this return true if you have https://google.com and http://google.com?
Should it?

     // TODO(ajwong): This does not correctly handle storage for isolated apps.
     session_storage_namespace =
         source_web_contents->GetController()
             .GetDefaultSessionStorageNamespace();
   }
 
-  return AddPrerender(ORIGIN_LINK_REL_PRERENDER,
+  if (PrerenderData* parent_prerender_data =
+          FindPrerenderDataForChildAndRoute(process_id, route_id)) {
+    // Instead of prerendering from inside of a running prerender, we will defer
+    // this request until its launcher is made visible.
+    if (PrerenderContents* contents = parent_prerender_data->contents_) {
+      pending_prerender_list_.push_back(
+          linked_ptr<PrerenderData>(new PrerenderData(this)));
+      PrerenderHandle* prerender_handle =
+          new PrerenderHandle(pending_prerender_list_.back().get());
+      contents->AddPendingPrerender(
+          prerender_handle->weak_ptr_factory_.GetWeakPtr(),
+          is_same_domain ? ORIGIN_LINK_REL_PRERENDER_SAMEDOMAIN
+                         : ORIGIN_LINK_REL_PRERENDER_CROSSDOMAIN,
+          url, referrer, size);
+      return prerender_handle;
+    }
+  }
+
+  return AddPrerender(is_same_domain ? ORIGIN_LINK_REL_PRERENDER_SAMEDOMAIN
+                                     : ORIGIN_LINK_REL_PRERENDER_CROSSDOMAIN,
                       process_id, url, referrer, size,
                       session_storage_namespace);
 #endif
 }
 
 PrerenderHandle* PrerenderManager::AddPrerenderFromOmnibox(
     const GURL& url,
     SessionStorageNamespace* session_storage_namespace,
     const gfx::Size& size) {
   if (!IsOmniboxEnabled(profile_))
@@ skipping 239 matching lines @@
     WebContents* web_contents,
     const GURL& url) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   PrerenderManager* prerender_manager =
       PrerenderManagerFactory::GetForProfile(
           Profile::FromBrowserContext(web_contents->GetBrowserContext()));
   if (!prerender_manager)
     return;
   if (!prerender_manager->IsEnabled())
     return;
-  Origin prerender_origin = ORIGIN_NONE;
+  const Origin* prerender_origin = NULL;
   bool was_prerender =
       prerender_manager->IsWebContentsPrerendered(web_contents,
                                                   &prerender_origin);
   bool was_complete_prerender = was_prerender ||
       prerender_manager->WouldWebContentsBePrerendered(web_contents);
   if (prerender_manager->IsWebContentsPrerendering(web_contents)) {
     prerender_manager->histograms_->RecordPageLoadTimeNotSwappedIn(
-        prerender_origin, perceived_page_load_time, url);
+        was_prerender ? *prerender_origin : ORIGIN_NONE,
+        perceived_page_load_time, url);
   } else {
     prerender_manager->histograms_->RecordPerceivedPageLoadTime(
-        prerender_origin, perceived_page_load_time, was_prerender,
-        was_complete_prerender, url);
-    if (was_prerender) {
-      prerender_manager->histograms_->RecordPercentLoadDoneAtSwapin(
-          prerender_origin, fraction_plt_elapsed_at_swap_in);
-    }
+        was_prerender ? *prerender_origin : ORIGIN_NONE,
+        perceived_page_load_time, was_prerender, was_complete_prerender, url);
+    prerender_manager->histograms_->RecordPercentLoadDoneAtSwapin(
+        was_prerender ? *prerender_origin : ORIGIN_NONE,
+        fraction_plt_elapsed_at_swap_in);
     if (prerender_manager->local_predictor_.get()) {
       prerender_manager->local_predictor_->
           OnPLTEventForURL(url, perceived_page_load_time);
     }
   }
 }
 
 void PrerenderManager::RecordFractionPixelsFinalAtSwapin(
     content::WebContents* web_contents,
     double fraction) {
-  Origin origin = ORIGIN_NONE;
+  const Origin* origin = NULL;
   IsWebContentsPrerendered(web_contents, &origin);
-  histograms_->RecordFractionPixelsFinalAtSwapin(origin, fraction);
+  histograms_->RecordFractionPixelsFinalAtSwapin(
+      origin ? *origin : ORIGIN_NONE, fraction);
 }
 
 void PrerenderManager::set_enabled(bool enabled) {
   DCHECK(CalledOnValidThread());
   enabled_ = enabled;
 }
 
 // static
 bool PrerenderManager::IsPrefetchEnabled() {
   return is_prefetch_enabled_;
@@ skipping 121 matching lines @@
     if (first_time) {
       it->second = false;
     } else {
       would_be_prerendered_map_.erase(it);
     }
   }
 }
 
 bool PrerenderManager::IsWebContentsPrerendered(
     content::WebContents* web_contents,
-    Origin* origin) const {
+    const Origin** origin) const {
   DCHECK(CalledOnValidThread());
   base::hash_map<content::WebContents*, PrerenderedWebContentsData>::
       const_iterator it = prerendered_web_contents_data_.find(web_contents);
   if (it == prerendered_web_contents_data_.end()) {
     if (origin)
-      *origin = ORIGIN_NONE;
+      *origin = NULL;

[dominich, 2012/10/08 17:03:04]

this could be ORIGIN_MAX. Or a new ORIGIN_INVALID (== -1)

     return false;
   }
   if (origin)
-    *origin = it->second.origin;
+    *origin = &it->second.origin;

[dominich, 2012/10/08 17:03:04]

this could lead to lifetime issues depending on what the caller does with it.

   return true;
 }
 
 bool PrerenderManager::WouldWebContentsBePrerendered(
     WebContents* web_contents) const {
   DCHECK(CalledOnValidThread());
   return would_be_prerendered_map_.count(web_contents) > 0;
 }
 
 bool PrerenderManager::HasRecentlyBeenNavigatedTo(Origin origin,
@@ skipping 235 matching lines @@
     int process_id,
     const GURL& url_arg,
     const content::Referrer& referrer,
     const gfx::Size& size,
     SessionStorageNamespace* session_storage_namespace) {
   DCHECK(CalledOnValidThread());
 
   if (!IsEnabled())
     return NULL;
 
-  if (origin == ORIGIN_LINK_REL_PRERENDER &&
+  if ((origin == ORIGIN_LINK_REL_PRERENDER_CROSSDOMAIN ||
+       origin == ORIGIN_LINK_REL_PRERENDER_SAMEDOMAIN) &&
       IsGoogleSearchResultURL(referrer.url)) {
     origin = ORIGIN_GWS_PRERENDER;
   }
 
   DeleteOldEntries();
   DeletePendingDeleteEntries();
 
   GURL url = url_arg;
   GURL alias_url;
   uint8 experiment = GetQueryStringBasedExperiment(url_arg);
@@ skipping 362 matching lines @@
   if (!render_process_host || !render_process_host->GetBrowserContext())
     return NULL;
   Profile* profile = Profile::FromBrowserContext(
       render_process_host->GetBrowserContext());
   if (!profile)
     return NULL;
   return PrerenderManagerFactory::GetInstance()->GetForProfile(profile);
 }
 
 }  // namespace prerender