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

Side by Side Diff: chrome/browser/metrics/chrome_metrics_service_client.cc

Issue 2918533003: Send metrics with embedded system profiles after system startup. (Closed)
Patch Set: fixed tests on mac Created 3 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/metrics/chrome_metrics_service_client.h" 5 #include "chrome/browser/metrics/chrome_metrics_service_client.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <set> 9 #include <set>
10 #include <utility> 10 #include <utility>
(...skipping 159 matching lines...) Expand 10 before | Expand all | Expand 10 after
170 #endif 170 #endif
171 } 171 }
172 172
173 // Constructs the name of a persistent metrics file from a directory and metrics 173 // Constructs the name of a persistent metrics file from a directory and metrics
174 // name, and either registers that file as associated with a previous run if 174 // name, and either registers that file as associated with a previous run if
175 // metrics reporting is enabled, or deletes it if not. 175 // metrics reporting is enabled, or deletes it if not.
176 void RegisterOrRemovePreviousRunMetricsFile( 176 void RegisterOrRemovePreviousRunMetricsFile(
177 bool metrics_reporting_enabled, 177 bool metrics_reporting_enabled,
178 const base::FilePath& dir, 178 const base::FilePath& dir,
179 base::StringPiece metrics_name, 179 base::StringPiece metrics_name,
180 bool require_internal_profile,
180 scoped_refptr<base::TaskRunner> task_runner, 181 scoped_refptr<base::TaskRunner> task_runner,
181 metrics::FileMetricsProvider* file_metrics_provider) { 182 metrics::FileMetricsProvider* file_metrics_provider) {
182 base::FilePath metrics_file; 183 base::FilePath metrics_file;
183 base::GlobalHistogramAllocator::ConstructFilePaths( 184 base::GlobalHistogramAllocator::ConstructFilePaths(
184 dir, metrics_name, &metrics_file, nullptr, nullptr); 185 dir, metrics_name, &metrics_file, nullptr, nullptr);
185 186
186 if (metrics_reporting_enabled) { 187 if (metrics_reporting_enabled) {
187 // Enable reading any existing saved metrics. 188 // Enable reading any existing saved metrics.
188 file_metrics_provider->RegisterSource( 189 file_metrics_provider->RegisterSource(
189 metrics_file, 190 metrics_file,
190 metrics::FileMetricsProvider::SOURCE_HISTOGRAMS_ATOMIC_FILE, 191 metrics::FileMetricsProvider::SOURCE_HISTOGRAMS_ATOMIC_FILE,
191 metrics::FileMetricsProvider::ASSOCIATE_PREVIOUS_RUN, metrics_name); 192 require_internal_profile
193 ? metrics::FileMetricsProvider::ASSOCIATE_INTERNAL_PROFILE
194 : metrics::FileMetricsProvider::
195 ASSOCIATE_INTERNAL_PROFILE_OR_PREVIOUS_RUN,
196 metrics_name);
192 } else { 197 } else {
193 // When metrics reporting is not enabled, any existing file should be 198 // When metrics reporting is not enabled, any existing file should be
194 // deleted in order to preserve user privacy. 199 // deleted in order to preserve user privacy.
195 task_runner->PostTask(FROM_HERE, 200 task_runner->PostTask(FROM_HERE,
196 base::BindOnce(base::IgnoreResult(&base::DeleteFile), 201 base::BindOnce(base::IgnoreResult(&base::DeleteFile),
197 metrics_file, /*recursive=*/false)); 202 metrics_file, /*recursive=*/false));
198 } 203 }
199 } 204 }
200 205
201 std::unique_ptr<metrics::FileMetricsProvider> CreateFileMetricsProvider( 206 std::unique_ptr<metrics::FileMetricsProvider> CreateFileMetricsProvider(
(...skipping 14 matching lines...) Expand all
216 if (base::PathService::Get(chrome::DIR_USER_DATA, &user_data_dir)) { 221 if (base::PathService::Get(chrome::DIR_USER_DATA, &user_data_dir)) {
217 // Reporting of persistent histograms from last session is controlled by 222 // Reporting of persistent histograms from last session is controlled by
218 // a feature param. TODO(bcwhite): The current default is not to upload 223 // a feature param. TODO(bcwhite): The current default is not to upload
219 // until some issues are resolved. See crbug.com/706422 for details. 224 // until some issues are resolved. See crbug.com/706422 for details.
220 std::string send_unreported = base::GetFieldTrialParamValueByFeature( 225 std::string send_unreported = base::GetFieldTrialParamValueByFeature(
221 base::kPersistentHistogramsFeature, "send_unreported_metrics"); 226 base::kPersistentHistogramsFeature, "send_unreported_metrics");
222 bool report_previous_persistent_histograms = 227 bool report_previous_persistent_histograms =
223 metrics_reporting_enabled && (send_unreported == "yes"); 228 metrics_reporting_enabled && (send_unreported == "yes");
224 RegisterOrRemovePreviousRunMetricsFile( 229 RegisterOrRemovePreviousRunMetricsFile(
225 report_previous_persistent_histograms, user_data_dir, 230 report_previous_persistent_histograms, user_data_dir,
226 ChromeMetricsServiceClient::kBrowserMetricsName, task_runner, 231 ChromeMetricsServiceClient::kBrowserMetricsName, true, task_runner,
227 file_metrics_provider.get()); 232 file_metrics_provider.get());
228 233
229 // Register the Crashpad metrics files. 234 // Register the Crashpad metrics files.
230 // Register the data from the previous run if crashpad_handler didn't exit 235 // Register the data from the previous run if crashpad_handler didn't exit
231 // cleanly. 236 // cleanly.
232 RegisterOrRemovePreviousRunMetricsFile( 237 RegisterOrRemovePreviousRunMetricsFile(
233 metrics_reporting_enabled, user_data_dir, 238 metrics_reporting_enabled, user_data_dir,
234 kCrashpadHistogramAllocatorName, task_runner, 239 kCrashpadHistogramAllocatorName, false, task_runner,
235 file_metrics_provider.get()); 240 file_metrics_provider.get());
236 if (metrics_reporting_enabled) { 241 if (metrics_reporting_enabled) {
237 base::FilePath active_path; 242 base::FilePath active_path;
238 base::GlobalHistogramAllocator::ConstructFilePaths( 243 base::GlobalHistogramAllocator::ConstructFilePaths(
239 user_data_dir, kCrashpadHistogramAllocatorName, nullptr, &active_path, 244 user_data_dir, kCrashpadHistogramAllocatorName, nullptr, &active_path,
240 nullptr); 245 nullptr);
241 // Register data that will be populated for the current run. "Active" 246 // Register data that will be populated for the current run. "Active"
242 // files need an empty "prefs_key" because they update the file itself. 247 // files need an empty "prefs_key" because they update the file itself.
243 file_metrics_provider->RegisterSource( 248 file_metrics_provider->RegisterSource(
244 active_path, 249 active_path,
(...skipping 748 matching lines...) Expand 10 before | Expand all | Expand 10 after
993 ukm_service_->Purge(); 998 ukm_service_->Purge();
994 ukm_service_->ResetClientId(); 999 ukm_service_->ResetClientId();
995 } 1000 }
996 // Signal service manager to enable/disable UKM based on new state. 1001 // Signal service manager to enable/disable UKM based on new state.
997 UpdateRunningServices(); 1002 UpdateRunningServices();
998 } 1003 }
999 1004
1000 bool ChromeMetricsServiceClient::IsHistorySyncEnabledOnAllProfiles() { 1005 bool ChromeMetricsServiceClient::IsHistorySyncEnabledOnAllProfiles() {
1001 return SyncDisableObserver::IsHistorySyncEnabledOnAllProfiles(); 1006 return SyncDisableObserver::IsHistorySyncEnabledOnAllProfiles();
1002 } 1007 }
OLDNEW
« no previous file with comments | « no previous file | components/metrics/file_metrics_provider.h » ('j') | components/metrics/metrics_log.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698