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

Unified Diff: chrome/browser/chrome_browser_main.cc

Issue 81603002: Send UMA stability stats in a separate UMA log on startup. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 7 years 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 | « no previous file | chrome/browser/metrics/metrics_service.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chrome_browser_main.cc
===================================================================
--- chrome/browser/chrome_browser_main.cc (revision 240851)
+++ chrome/browser/chrome_browser_main.cc (working copy)
@@ -589,12 +589,14 @@
// Initialize FieldTrialList to support FieldTrials that use one-time
// randomization.
MetricsService* metrics = browser_process_->metrics_service();
- bool metrics_reporting_enabled = IsMetricsReportingEnabled();
- if (metrics_reporting_enabled)
+ MetricsService::ReportingState reporting_state =
+ IsMetricsReportingEnabled() ? MetricsService::REPORTING_ENABLED :
+ MetricsService::REPORTING_DISABLED;
+ if (reporting_state == MetricsService::REPORTING_ENABLED)
metrics->ForceClientIdCreation(); // Needed below.
field_trial_list_.reset(
new base::FieldTrialList(
- metrics->CreateEntropyProvider(metrics_reporting_enabled).release()));
+ metrics->CreateEntropyProvider(reporting_state).release()));
const CommandLine* command_line = CommandLine::ForCurrentProcess();
if (command_line->HasSwitch(switches::kEnableBenchmarking))
@@ -634,6 +636,9 @@
// Even though base::Bind does AddRef and Release, the object will not be
// deleted after the Task is executed.
field_trial_synchronizer_ = new FieldTrialSynchronizer();
+
+ // Now that field trials have been created, initializes metrics recording.
+ metrics->InitializeMetricsRecordingState(reporting_state);
}
// ChromeBrowserMainParts: |SetupMetricsAndFieldTrials()| related --------------
« no previous file with comments | « no previous file | chrome/browser/metrics/metrics_service.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698