Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(499)

Unified Diff: chrome/browser/prerender/prerender_manager.cc

Issue 8602005: Adding histograms for better Omnibox prerender performance tracking. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Move prerender histogram access to prerender_histogram.cc Created 9 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/prerender/prerender_manager.cc
diff --git a/chrome/browser/prerender/prerender_manager.cc b/chrome/browser/prerender/prerender_manager.cc
index 18093a932c96f1e72d12bfa15f0c789a2a624cd6..aabe51e21f89595271d214371f0994acc72e273b 100644
--- a/chrome/browser/prerender/prerender_manager.cc
+++ b/chrome/browser/prerender/prerender_manager.cc
@@ -358,6 +358,8 @@ bool PrerenderManager::AddPrerenderFromOmnibox(
break;
};
+ histograms_->RecordPrerenderFromOmnibox();
+
return AddPrerender(origin, std::make_pair(-1, -1), url, GURL(),
session_storage_namespace);
}
@@ -646,6 +648,8 @@ bool PrerenderManager::MaybeUsePrerenderedPage(TabContents* tab_contents,
histograms_->RecordPerSessionCount(++prerenders_per_session_count_);
prerender_contents->set_final_status(FINAL_STATUS_USED);
+ histograms_->RecordOmniboxUsedPrerender(prerender_contents->origin());
+
new_render_view_host->Send(
new ChromeViewMsg_SetIsPrerendering(new_render_view_host->routing_id(),
false));

Powered by Google App Engine
This is Rietveld 408576698