Spoken Feedback Browsertest: AddBookmark test and some structural changes

chrome/browser/chromeos/accessibility/spoken_feedback_browsertest.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <queue>
 
 #include "ash/shell.h"
 #include "base/command_line.h"
 #include "base/strings/string_util.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/chromeos/accessibility/accessibility_manager.h"
 #include "chrome/browser/chromeos/accessibility/speech_monitor.h"
 #include "chrome/browser/chromeos/login/ui/login_display_host.h"
 #include "chrome/browser/chromeos/login/ui/login_display_host_impl.h"
 #include "chrome/browser/chromeos/login/ui/webui_login_view.h"
 #include "chrome/browser/chromeos/login/users/user_manager.h"
 #include "chrome/browser/chromeos/profiles/profile_helper.h"
 #include "chrome/browser/extensions/api/braille_display_private/stub_braille_controller.h"
 #include "chrome/browser/speech/tts_controller.h"
 #include "chrome/browser/speech/tts_platform.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_commands.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/test/base/in_process_browser_test.h"
+#include "chrome/test/base/interactive_test_utils.h"
 #include "chrome/test/base/testing_profile.h"
 #include "chrome/test/base/ui_test_utils.h"
 #include "chromeos/chromeos_switches.h"
 #include "content/public/common/url_constants.h"
 #include "content/public/test/test_utils.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/base/test/ui_controls.h"
 #include "ui/views/widget/widget.h"
 
 using extensions::api::braille_display_private::StubBrailleController;
 
 namespace chromeos {
 
 //
-// Spoken feedback tests in a normal browser window.
+// Spoken feedback tests only in a logged in user's window.
+//
+
+class LoggedInSpokenFeedbackTest : public InProcessBrowserTest {
+ protected:
+  LoggedInSpokenFeedbackTest() {}
+  virtual ~LoggedInSpokenFeedbackTest() {}
+
+  virtual void SetUpInProcessBrowserTestFixture() OVERRIDE {
+    AccessibilityManager::SetBrailleControllerForTest(&braille_controller_);
+  }
+
+  virtual void CleanUpOnMainThread() OVERRIDE {
+    AccessibilityManager::SetBrailleControllerForTest(NULL);
+  }
+
+  void SendKeyPress(ui::KeyboardCode key) {
+    gfx::NativeWindow root_window =
+        ash::Shell::GetInstance()->GetPrimaryRootWindow();
+    ASSERT_TRUE(
+        ui_test_utils::SendKeyPressToWindowSync(
+            root_window, key, false, false, false, false));
+  }
+
+ private:
+  StubBrailleController braille_controller_;
+  DISALLOW_COPY_AND_ASSIGN(LoggedInSpokenFeedbackTest);
+};
+
+IN_PROC_BROWSER_TEST_F(LoggedInSpokenFeedbackTest, AddBookmark) {
+  EXPECT_FALSE(AccessibilityManager::Get()->IsSpokenFeedbackEnabled());
+
+  SpeechMonitor monitor;
+  AccessibilityManager::Get()->EnableSpokenFeedback(
+      true, ash::A11Y_NOTIFICATION_NONE);
+  EXPECT_TRUE(monitor.SkipChromeVoxEnabledMessage());
+
+  chrome::ExecuteCommand(browser(), IDC_SHOW_BOOKMARK_BAR);
+
+  // Create a bookmark with title "foo".
+  chrome::ExecuteCommand(browser(), IDC_BOOKMARK_PAGE);
+  EXPECT_EQ("Bookmark added!,", monitor.GetNextUtterance());
+  EXPECT_EQ("about blank,", monitor.GetNextUtterance());
+  EXPECT_EQ("Bookmark name,", monitor.GetNextUtterance());
+  EXPECT_EQ("text box", monitor.GetNextUtterance());
+
+  SendKeyPress(ui::VKEY_F);
+  EXPECT_EQ("f", monitor.GetNextUtterance());
+  SendKeyPress(ui::VKEY_O);
+  EXPECT_EQ("o", monitor.GetNextUtterance());
+  SendKeyPress(ui::VKEY_O);
+  EXPECT_EQ("o", monitor.GetNextUtterance());
+
+  SendKeyPress(ui::VKEY_TAB);
+  EXPECT_EQ("Bookmarks bar,", monitor.GetNextUtterance());
+  EXPECT_EQ("Bookmark folder,", monitor.GetNextUtterance());
+  EXPECT_TRUE(MatchPattern(monitor.GetNextUtterance(), "combo box*"));
+
+  SendKeyPress(ui::VKEY_RETURN);
+
+  // Focus bookmarks bar and listen for "foo".
+  chrome::ExecuteCommand(browser(), IDC_FOCUS_BOOKMARKS);
+  while (true) {
+    std::string utterance = monitor.GetNextUtterance();
+    VLOG(0) << "Got utterance: " << utterance;
+    if (utterance == "Bookmarks,")
+      break;
+  }
+  EXPECT_EQ("foo,", monitor.GetNextUtterance());
+  EXPECT_EQ("button", monitor.GetNextUtterance());
+}
+
+//
+// Spoken feedback tests in both a logged in browser window and guest mode.
 //
 
 enum SpokenFeedbackTestVariant {
   kTestAsNormalUser,
   kTestAsGuestUser
 };
 
 class SpokenFeedbackTest
-    : public InProcessBrowserTest,
+    : public LoggedInSpokenFeedbackTest,
       public ::testing::WithParamInterface<SpokenFeedbackTestVariant> {
  protected:
   SpokenFeedbackTest() {}
   virtual ~SpokenFeedbackTest() {}
 
-  virtual void SetUpInProcessBrowserTestFixture() OVERRIDE {
-    AccessibilityManager::SetBrailleControllerForTest(&braille_controller_);
-  }
-
-  virtual void CleanUpOnMainThread() OVERRIDE {
-    AccessibilityManager::SetBrailleControllerForTest(NULL);
-  }
-
   virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
     if (GetParam() == kTestAsGuestUser) {
       command_line->AppendSwitch(chromeos::switches::kGuestSession);
       command_line->AppendSwitch(::switches::kIncognito);
       command_line->AppendSwitchASCII(chromeos::switches::kLoginProfile,
                                       "user");
       command_line->AppendSwitchASCII(chromeos::switches::kLoginUser,
                                       chromeos::UserManager::kGuestUserName);
     }
   }
-
- private:
-  StubBrailleController braille_controller_;
-  DISALLOW_COPY_AND_ASSIGN(SpokenFeedbackTest);
 };
 
 INSTANTIATE_TEST_CASE_P(
     TestAsNormalAndGuestUser,
     SpokenFeedbackTest,
     ::testing::Values(kTestAsNormalUser,
                       kTestAsGuestUser));
 
 IN_PROC_BROWSER_TEST_P(SpokenFeedbackTest, EnableSpokenFeedback) {
   EXPECT_FALSE(AccessibilityManager::Get()->IsSpokenFeedbackEnabled());
@@ skipping 12 matching lines @@
       true, ash::A11Y_NOTIFICATION_NONE);
   EXPECT_TRUE(monitor.SkipChromeVoxEnabledMessage());
 
   chrome::ExecuteCommand(browser(), IDC_FOCUS_TOOLBAR);
   // Might be "Google Chrome Toolbar" or "Chromium Toolbar".
   EXPECT_TRUE(MatchPattern(monitor.GetNextUtterance(), "*oolbar*"));
   EXPECT_EQ("Reload,", monitor.GetNextUtterance());
   EXPECT_EQ("button", monitor.GetNextUtterance());
 }
 
-// Disabled for flakiness: http://crbug.com/359204
-IN_PROC_BROWSER_TEST_P(SpokenFeedbackTest, DISABLED_TypeInOmnibox) {
+IN_PROC_BROWSER_TEST_P(SpokenFeedbackTest, TypeInOmnibox) {
   EXPECT_FALSE(AccessibilityManager::Get()->IsSpokenFeedbackEnabled());
 
   SpeechMonitor monitor;
   AccessibilityManager::Get()->EnableSpokenFeedback(
       true, ash::A11Y_NOTIFICATION_NONE);
   EXPECT_TRUE(monitor.SkipChromeVoxEnabledMessage());
 
+  // Wait for ChromeVox to finish speaking.
   chrome::ExecuteCommand(browser(), IDC_FOCUS_LOCATION);
-
-  gfx::NativeWindow window = ash::Shell::GetInstance()->GetPrimaryRootWindow();
-  ui_controls::SendKeyPress(window, ui::VKEY_X, false, false, false, false);
   while (true) {
     std::string utterance = monitor.GetNextUtterance();
     VLOG(0) << "Got utterance: " << utterance;
-    if (utterance == "x")
+    if (utterance == "text box")
       break;
   }
 
-  ui_controls::SendKeyPress(window, ui::VKEY_Y, false, false, false, false);
+  SendKeyPress(ui::VKEY_X);
+  EXPECT_EQ("x", monitor.GetNextUtterance());
+
+  SendKeyPress(ui::VKEY_Y);
   EXPECT_EQ("y", monitor.GetNextUtterance());
 
-  ui_controls::SendKeyPress(window, ui::VKEY_Z, false, false, false, false);
+  SendKeyPress(ui::VKEY_Z);
   EXPECT_EQ("z", monitor.GetNextUtterance());
 
-  ui_controls::SendKeyPress(window, ui::VKEY_BACK, false, false, false, false);
+  SendKeyPress(ui::VKEY_BACK);
   EXPECT_EQ("z", monitor.GetNextUtterance());
 }
 
 //
-// Spoken feedback tests that run in guest mode.
+// Spoken feedback tests that run only in guest mode.
 //
 
-class GuestSpokenFeedbackTest : public SpokenFeedbackTest {
+class GuestSpokenFeedbackTest : public LoggedInSpokenFeedbackTest {
  protected:
   GuestSpokenFeedbackTest() {}
   virtual ~GuestSpokenFeedbackTest() {}
 
   virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
     command_line->AppendSwitch(chromeos::switches::kGuestSession);
     command_line->AppendSwitch(::switches::kIncognito);
     command_line->AppendSwitchASCII(chromeos::switches::kLoginProfile, "user");
     command_line->AppendSwitchASCII(chromeos::switches::kLoginUser,
                                     chromeos::UserManager::kGuestUserName);
@@ skipping 53 matching lines @@
   gfx::NativeWindow window = widget->GetNativeWindow();
 
   SpeechMonitor monitor;
   AccessibilityManager::Get()->EnableSpokenFeedback(
       true, ash::A11Y_NOTIFICATION_NONE);
   EXPECT_TRUE(monitor.SkipChromeVoxEnabledMessage());
 
   EXPECT_EQ("Select your language:", monitor.GetNextUtterance());
   EXPECT_EQ("English ( United States)", monitor.GetNextUtterance());
   EXPECT_TRUE(MatchPattern(monitor.GetNextUtterance(), "Combo box * of *"));
-  ui_controls::SendKeyPress(window, ui::VKEY_TAB, false, false, false, false);
+  ASSERT_TRUE(
+      ui_test_utils::SendKeyPressToWindowSync(
+          window, ui::VKEY_TAB, false, false, false, false));
   EXPECT_EQ("Select your keyboard:", monitor.GetNextUtterance());
 }
 
 }  // namespace chromeos