[StopSyncInPausedState] Enable kStopSyncInPausedState by default

Bug: 938819, 906995
Change-Id: I41287062e67b4bac4b6eda3158a529b527e0c1bb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2083531
Commit-Queue: Maksim Moskvitin <mmoskvitin@google.com>
Reviewed-by: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#748235}

chrome/browser/sync/test/integration/single_client_user_events_sync_test.cc

@@ -225,6 +225,8 @@ IN_PROC_BROWSER_TEST_F(SingleClientUserEventsSyncTest,
 
   // Clear the "Sync paused" state again.
   GetClient(0)->ExitSyncPausedStateForPrimaryAccount();
+  // Once the auth error is gone, wait for Sync to start up again.
+  GetClient(0)->AwaitSyncSetupCompletion();
   ASSERT_TRUE(GetSyncService(0)->IsSyncFeatureActive());
 
   // Just checking that we don't see test_event isn't very convincing yet,

chrome/browser/ui/views/profiles/profile_menu_view_browsertest.cc

@@ -514,7 +514,7 @@ PROFILE_MENU_CLICK_TEST(kActionableItems_SyncPaused,
   sync_harness()->EnterSyncPausedStateForPrimaryAccount();
   // Check that the setup was successful.
   ASSERT_TRUE(identity_manager()->HasPrimaryAccount());
-  ASSERT_FALSE(sync_service()->HasDisableReason(
+  ASSERT_TRUE(sync_service()->HasDisableReason(
       syncer::SyncService::DISABLE_REASON_PAUSED));
 
   RunTest();

components/sync/driver/sync_driver_switches.cc

@@ -51,7 +51,7 @@ const base::Feature kSyncAllowWalletDataInTransportModeWithCustomPassphrase{
 
 // If enabled, the sync engine will be shut down in the "paused" state.
 const base::Feature kStopSyncInPausedState{"StopSyncInPausedState",
-                                           base::FEATURE_DISABLED_BY_DEFAULT};
+                                           base::FEATURE_ENABLED_BY_DEFAULT};
 
 // Enable USS implementation of Passwords datatype.
 const base::Feature kSyncUSSPasswords{"SyncUSSPasswords",