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

Unified Diff: chrome/browser/browser_main.cc

Issue 6340013: Add histogram to track prerender sessions (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix initialization Created 9 years, 11 months 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
« no previous file with comments | « chrome/browser/browser_main.h ('k') | chrome/browser/prerender/prerender_manager.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/browser_main.cc
diff --git a/chrome/browser/browser_main.cc b/chrome/browser/browser_main.cc
index df290e35c2fc677f1b34e26dcdb07809a2c3520d..f937f3ffc120ce88eedc008d101ca0f8642fd51b 100644
--- a/chrome/browser/browser_main.cc
+++ b/chrome/browser/browser_main.cc
@@ -56,6 +56,7 @@
#include "chrome/browser/prefs/browser_prefs.h"
#include "chrome/browser/prefs/pref_service.h"
#include "chrome/browser/prefs/pref_value_store.h"
+#include "chrome/browser/prerender/prerender_manager.h"
#include "chrome/browser/printing/cloud_print/cloud_print_proxy_service.h"
#include "chrome/browser/printing/print_dialog_cloud.h"
#include "chrome/browser/process_singleton.h"
@@ -224,7 +225,7 @@ void BrowserMainParts::SetupFieldTrials() {
ConnectionFieldTrial();
SocketTimeoutFieldTrial();
ProxyConnectionsFieldTrial();
- PrefetchFieldTrial();
+ PrefetchAndPrerenderFieldTrial();
SpdyFieldTrial();
ConnectBackupJobsFieldTrial();
}
@@ -429,7 +430,7 @@ void BrowserMainParts::SpdyFieldTrial() {
// --disable-content-prefetch are set, use those to determine if
// prefetch is enabled. Otherwise, randomly assign users to an A/B test for
// prefetching.
-void BrowserMainParts::PrefetchFieldTrial() {
+void BrowserMainParts::PrefetchAndPrerenderFieldTrial() {
if (parsed_command_line().HasSwitch(switches::kEnableContentPrefetch) ||
parsed_command_line().HasSwitch(switches::kEnablePagePrerender))
ResourceDispatcherHost::set_is_prefetch_enabled(true);
@@ -448,6 +449,17 @@ void BrowserMainParts::PrefetchFieldTrial() {
ResourceDispatcherHost::set_is_prefetch_enabled(
trial_grp == yes_prefetch_grp);
}
+
+ PrerenderManager::PrerenderManagerMode prerender_mode =
+ PrerenderManager::PRERENDER_MODE_DISABLED;
+ if (parsed_command_line().HasSwitch(switches::kEnablePagePrerender))
+ prerender_mode = PrerenderManager::PRERENDER_MODE_ENABLED;
+ else
+ prerender_mode = PrerenderManager::PRERENDER_MODE_DISABLED;
+ PrerenderManager::SetMode(prerender_mode);
+
+ UMA_HISTOGRAM_ENUMERATION("Prerender.Sessions", prerender_mode,
+ PrerenderManager::PRERENDER_MODE_MAX);
}
// If neither --enable-connect-backup-jobs or --disable-connect-backup-jobs is
« no previous file with comments | « chrome/browser/browser_main.h ('k') | chrome/browser/prerender/prerender_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698