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

Unified Diff: chrome/browser/chrome_browser_main.cc

Issue 2380043002: Do not register synthetic field trial before initializing TaskScheduler. (Closed)
Patch Set: rebase Created 4 years, 3 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/chrome_browser_main.cc
diff --git a/chrome/browser/chrome_browser_main.cc b/chrome/browser/chrome_browser_main.cc
index 426d19ada52e7d3ebae8f225bb70475dd68ed57f..ee3ff2b369b8934b142e6a0e5f367d82b377dd8a 100644
--- a/chrome/browser/chrome_browser_main.cc
+++ b/chrome/browser/chrome_browser_main.cc
@@ -890,15 +890,6 @@ void ChromeBrowserMainParts::SetupFieldTrials() {
// deleted after the Task is executed.
field_trial_synchronizer_ = new FieldTrialSynchronizer();
- // Register a synthetic field trial for the sampling profiler configuration
- // that was already chosen.
- std::string trial_name, group_name;
- if (sampling_profiler_config_.GetSyntheticFieldTrial(&trial_name,
- &group_name)) {
- ChromeMetricsServiceAccessor::RegisterSyntheticFieldTrial(trial_name,
- group_name);
- }
-
#if defined(OS_WIN) || defined(OS_MACOSX) || \
(defined(OS_LINUX) && !defined(OS_CHROMEOS))
metrics::DesktopSessionDurationTracker::Initialize();
@@ -947,6 +938,15 @@ void ChromeBrowserMainParts::StartMetricsRecording() {
g_browser_process->metrics_service()->CheckForClonedInstall(
BrowserThread::GetTaskRunnerForThread(BrowserThread::FILE));
+ // Register a synthetic field trial for the sampling profiler configuration
+ // that was already chosen.
+ std::string trial_name, group_name;
+ if (sampling_profiler_config_.GetSyntheticFieldTrial(&trial_name,
+ &group_name)) {
+ ChromeMetricsServiceAccessor::RegisterSyntheticFieldTrial(trial_name,
+ group_name);
+ }
+
g_browser_process->GetMetricsServicesManager()->UpdateUploadPermissions(true);
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698