diff options
| author | petersont@google.com <petersont@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-09-13 00:06:01 +0000 |
|---|---|---|
| committer | petersont@google.com <petersont@google.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2008-09-13 00:06:01 +0000 |
| commit | f4f2df8024d7adf583a61b742a32eaa17522f154 (patch) | |
| tree | b26aaab1de5047302ff5289313390d93ea6d9468 /chrome/browser/browser_main.cc | |
| parent | ff2a3985d4ec40ce537cf0919cd99835fe12e0d3 (diff) | |
| download | chromium_src-f4f2df8024d7adf583a61b742a32eaa17522f154.zip chromium_src-f4f2df8024d7adf583a61b742a32eaa17522f154.tar.gz chromium_src-f4f2df8024d7adf583a61b742a32eaa17522f154.tar.bz2 | |
Added routines to MetricsService to parse the server response and respond to the request to cease uploading with some probability.
Review URL: http://codereview.chromium.org/1633
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@2151 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/browser_main.cc')
| -rw-r--r-- | chrome/browser/browser_main.cc | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/chrome/browser/browser_main.cc b/chrome/browser/browser_main.cc index e574ef8a..764e733 100644 --- a/chrome/browser/browser_main.cc +++ b/chrome/browser/browser_main.cc @@ -468,8 +468,12 @@ int BrowserMain(CommandLine &parsed_command_line, int show_command, } metrics = browser_process->metrics_service(); DCHECK(metrics); - // Start user experience metrics recording, if enabled. - metrics->SetRecording(local_state->GetBoolean(prefs::kMetricsIsRecording)); + // If the user permits metrics reporting with the checkbox in the + // prefs, we turn on recording. + bool enabled = local_state->GetBoolean(prefs::kMetricsReportingEnabled); + metrics->SetUserPermitsUpload(enabled); + if (enabled) + metrics->Start(); } InstallJankometer(parsed_command_line); @@ -490,7 +494,7 @@ int BrowserMain(CommandLine &parsed_command_line, int show_command, } if (metrics) - metrics->SetRecording(false); // Force persistent save. + metrics->Stop(); // browser_shutdown takes care of deleting browser_process, so we need to // release it. |