Reland "oobe a11y: Add a test to verify profile switch during user login."

This is a reland of 0c9cf96c

Original change's description:
> oobe a11y: Add a test to verify profile switch during user login.
>
> Profile switches between signin and user during the login. Accessibility
> manager should correctly switch the profile.
>
> Also reeanbled OobeSpokenFeedbackTest on MSAN. It's already filtered out
> by chromeos.msan.browser_tests.oobe_(positive|negative).filter files.
>
> Bug: 1108996
> Change-Id: Ic586f7172e26bd965c42a9a566005fd872a69697
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2474696
> Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org>
> Reviewed-by: David Tseng <dtseng@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#817861}

Bug: 1108996, 1139530
Change-Id: Iee0384a2293f7d62853c22201171b786f8857fa7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2485218
Auto-Submit: Roman Sorokin [CET] <rsorokin@chromium.org>
Reviewed-by: David Tseng <dtseng@chromium.org>
Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#818885}

chrome/browser/chromeos/accessibility/spoken_feedback_browsertest.cc

@@ -23,7 +23,12 @@
 #include "build/build_config.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/chromeos/accessibility/accessibility_manager.h"
+#include "chrome/browser/chromeos/login/screens/sync_consent_screen.h"
+#include "chrome/browser/chromeos/login/test/device_state_mixin.h"
+#include "chrome/browser/chromeos/login/test/login_manager_mixin.h"
 #include "chrome/browser/chromeos/login/test/oobe_base_test.h"
+#include "chrome/browser/chromeos/profiles/profile_helper.h"
+#include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/browser/ui/ash/launcher/chrome_launcher_controller.h"
 #include "chrome/browser/ui/aura/accessibility/automation_manager_aura.h"
 #include "chrome/browser/ui/browser.h"
@@ -33,6 +38,7 @@
 #include "chrome/common/extensions/extension_constants.h"
 #include "chrome/test/base/interactive_test_utils.h"
 #include "chrome/test/base/ui_test_utils.h"
+#include "chromeos/constants/chromeos_features.h"
 #include "chromeos/constants/chromeos_switches.h"
 #include "components/user_manager/user_names.h"
 #include "content/public/browser/browser_task_traits.h"
@@ -936,13 +942,7 @@ class OobeSpokenFeedbackTest : public OobeBaseTest {
   DISALLOW_COPY_AND_ASSIGN(OobeSpokenFeedbackTest);
 };
 
-#if defined(MEMORY_SANITIZER)
-// Times out under MSan: https://crbug.com/1071693
-#define MAYBE_SpokenFeedbackInOobe DISABLED_SpokenFeedbackInOobe
-#else
-#define MAYBE_SpokenFeedbackInOobe SpokenFeedbackInOobe
-#endif
-IN_PROC_BROWSER_TEST_F(OobeSpokenFeedbackTest, MAYBE_SpokenFeedbackInOobe) {
+IN_PROC_BROWSER_TEST_F(OobeSpokenFeedbackTest, SpokenFeedbackInOobe) {
   ui_controls::EnableUIControls();
   ASSERT_FALSE(AccessibilityManager::Get()->IsSpokenFeedbackEnabled());
   AccessibilityManager::Get()->EnableSpokenFeedback(true);
@@ -960,4 +960,45 @@ IN_PROC_BROWSER_TEST_F(OobeSpokenFeedbackTest, MAYBE_SpokenFeedbackInOobe) {
   sm_.Replay();
 }
 
+class SigninToUserProfileSwitchTest : public OobeSpokenFeedbackTest {
+ public:
+  // OobeSpokenFeedbackTest:
+  void SetUpCommandLine(base::CommandLine* command_line) override {
+    OobeSpokenFeedbackTest::SetUpCommandLine(command_line);
+    // Force the help app to launch in the background.
+    command_line->AppendSwitch(switches::kForceFirstRunUI);
+  }
+
+ protected:
+  LoginManagerMixin login_manager_{&mixin_host_};
+  DeviceStateMixin device_state_{
+      &mixin_host_, DeviceStateMixin::State::OOBE_COMPLETED_UNOWNED};
+};
+
+// Verifies that spoken feedback correctly handles profile switch (signin ->
+// user) and announces the sync consent screen correctly.
+IN_PROC_BROWSER_TEST_F(SigninToUserProfileSwitchTest, LoginAsNewUser) {
+  // Force sync screen.
+  auto reset = SyncConsentScreen::ForceBrandedBuildForTesting(true);
+  AccessibilityManager::Get()->EnableSpokenFeedback(true);
+  sm_.ExpectSpeechPattern("*");
+
+  sm_.Call([this]() {
+    ASSERT_EQ(chromeos::AccessibilityManager::Get()->profile(),
+              ProfileHelper::GetSigninProfile());
+    login_manager_.LoginAsNewRegularUser();
+  });
+
+  std::string button_title =
+      features::IsSplitSettingsSyncEnabled() ? "Got it" : "Accept and continue";
+  sm_.ExpectSpeech(button_title);
+
+  // Check that profile switched to the active user.
+  sm_.Call([]() {
+    ASSERT_EQ(chromeos::AccessibilityManager::Get()->profile(),
+              ProfileManager::GetActiveUserProfile());
+  });
+  sm_.Replay();
+}
+
 }  // namespace chromeos

testing/buildbot/filters/chromeos.msan.browser_tests.oobe_negative.filter

@@ -116,6 +116,7 @@
 -SAMLPolicyTest.*
 -SamlTest.*
 -SecurityTokenSamlTest.*
+-SigninToUserProfileSwitchTest.*
 -SiteIsolationFlagHandlingTest.*
 -StartupBrowserWindowLaunchSuppressedTest.*
 -SyncConsentRecorderTest.*

testing/buildbot/filters/chromeos.msan.browser_tests.oobe_positive.filter

@@ -116,6 +116,7 @@ SAMLEnrollmentTest.*
 SAMLPolicyTest.*
 SamlTest.*
 SecurityTokenSamlTest.*
+SigninToUserProfileSwitchTest.*
 SiteIsolationFlagHandlingTest.*
 StartupBrowserWindowLaunchSuppressedTest.*
 SyncConsentRecorderTest.*