Commit c91fe1d6 authored by Olivier Li's avatar Olivier Li Committed by Commit Bot

Enable ProfileSyncServiceUsesThreadPool by default.

After we previously run as an inconclusive beta trial by itself we
grouped it with the HistoryService ThreadPool trial to try and coax a larger
impact from the ThreadPool migration. This meant groups ran with either both
services on the ThreadPool, none or only HistoryService (status quo).

28 day aggregation trial in beta showed no likely negative impact. Crashes
seemed slightly elevated in the group when running both the HistoryService
and the ProfileSyncService on the ThreadPool but closer examination suggests
this is simply noise since all top-crashers are the same.

Bug: 1014464
Change-Id: I3a1604413fcf1f4b5350c3b9161251f46a449865
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2198932
Commit-Queue: Oliver Li <olivierli@google.com>
Reviewed-by: default avatarMikel Astiz <mastiz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#768548}
parent 15d9842b
......@@ -66,6 +66,6 @@ const base::Feature kSyncDeviceInfoInTransportMode{
// Enables the running of backend ProfileSyncService tasks on the ThreadPool.
const base::Feature kProfileSyncServiceUsesThreadPool{
"ProfileSyncServiceUsesThreadPool", base::FEATURE_DISABLED_BY_DEFAULT};
"ProfileSyncServiceUsesThreadPool", base::FEATURE_ENABLED_BY_DEFAULT};
} // namespace switches
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment