Reland: Make ChromeVox Next the default ChromeVox experience

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/common/accelerators/accelerator_controller.h"
 #include "ash/common/accelerators/accelerator_table.h"
 #include "ash/common/accessibility_types.h"
 #include "ash/common/system/tray/system_tray.h"
@@ skipping 69 matching lines @@
   void SendKeyPressWithControl(ui::KeyboardCode key) {
     ASSERT_NO_FATAL_FAILURE(ASSERT_TRUE(ui_test_utils::SendKeyPressToWindowSync(
         nullptr, key, true, false, false, false)));
   }
 
   void SendKeyPressWithSearchAndShift(ui::KeyboardCode key) {
     ASSERT_NO_FATAL_FAILURE(ASSERT_TRUE(ui_test_utils::SendKeyPressToWindowSync(
         nullptr, key, false, true, false, true)));
   }
 
+  void SendKeyPressWithSearch(ui::KeyboardCode key) {
+    ASSERT_NO_FATAL_FAILURE(ASSERT_TRUE(ui_test_utils::SendKeyPressToWindowSync(
+        nullptr, key, false, false, false, true)));
+  }
+
   void RunJavaScriptInChromeVoxBackgroundPage(const std::string& script) {
     extensions::ExtensionHost* host =
         extensions::ProcessManager::Get(browser()->profile())
             ->GetBackgroundHostForExtension(
                 extension_misc::kChromeVoxExtensionId);
     CHECK(content::ExecuteScript(host->host_contents(), script));
   }
 
   void SimulateTouchScreenInChromeVox() {
     // ChromeVox looks at whether 'ontouchstart' exists to know whether
@@ skipping 13 matching lines @@
     // Playing earcons from within a test is not only annoying if you're
     // running the test locally, but seems to cause crashes
     // (http://crbug.com/396507). Work around this by just telling
     // ChromeVox to not ever play earcons (prerecorded sound effects).
     RunJavaScriptInChromeVoxBackgroundPage(
         "cvox.ChromeVox.earcons.playEarcon = function() {};");
   }
 
   void EnableChromeVox() {
     // Test setup.
-    // Enable ChromeVox, skip welcome message, and disable earcons.
+    // Enable ChromeVox, skip welcome message/notification, and disable earcons.
     ASSERT_FALSE(AccessibilityManager::Get()->IsSpokenFeedbackEnabled());
 
     AccessibilityManager::Get()->EnableSpokenFeedback(
         true, ash::A11Y_NOTIFICATION_NONE);
     EXPECT_TRUE(speech_monitor_.SkipChromeVoxEnabledMessage());
     DisableEarcons();
-  }
 
-  void LoadChromeVoxAndThenNavigateToURL(const GURL& url) {
-    // The goal of this helper function is to avoid race conditions between
-    // the page loading and the ChromeVox extension loading and fully
-    // initializing. To do this, we first load a test url that repeatedly
-    // asks ChromeVox to speak 'ready', then we load ChromeVox and block
-    // until we get that 'ready' speech.
-
-    ui_test_utils::NavigateToURL(
-        browser(),
-        GURL("data:text/html;charset=utf-8,"
-             "<script>"
-             "window.setInterval(function() {"
-             "  try {"
-             "    cvox.Api.speak('ready');"
-             "  } catch (e) {}"
-             "}, 100);"
-             "</script>"));
-    EXPECT_FALSE(AccessibilityManager::Get()->IsSpokenFeedbackEnabled());
-    AccessibilityManager::Get()->EnableSpokenFeedback(
-        true, ash::A11Y_NOTIFICATION_NONE);
-
-    // Block until we get "ready".
-    while (speech_monitor_.GetNextUtterance() != "ready") {
-    }
-
-    // Now load the requested url.
-    ui_test_utils::NavigateToURL(browser(), url);
+    // Ensure we skip all other startup messages by sending an additional speak
+    // call.
+    RunJavaScriptInChromeVoxBackgroundPage(
+        "cvox.ChromeVox.tts.speak('ready to test');");
+    EXPECT_TRUE(speech_monitor_.SkipChromeVoxMessage("ready to test"));
   }
 
   void PressRepeatedlyUntilUtterance(ui::KeyboardCode key,
                                      const std::string& expected_utterance) {
     // This helper function is needed when you want to poll for something
     // that happens asynchronously. Keep pressing |key|, until
     // the speech feedback that follows is |expected_utterance|.
     // Note that this doesn't work if pressing that key doesn't speak anything
     // at all before the asynchronous event occurred.
     while (true) {
@@ skipping 133 matching lines @@
 // AudioOutputResampler::Shutdown, see crbug.com/630031.
 IN_PROC_BROWSER_TEST_P(SpokenFeedbackTest, DISABLED_EnableSpokenFeedback) {
   EnableChromeVox();
 }
 
 IN_PROC_BROWSER_TEST_P(SpokenFeedbackTest, FocusToolbar) {
   EnableChromeVox();
   chrome::ExecuteCommand(browser(), IDC_FOCUS_TOOLBAR);
   EXPECT_EQ("Reload", speech_monitor_.GetNextUtterance());
   EXPECT_EQ("Button", speech_monitor_.GetNextUtterance());
-  EXPECT_EQ("main", speech_monitor_.GetNextUtterance());
-  EXPECT_EQ("Tool bar", speech_monitor_.GetNextUtterance());
-  EXPECT_TRUE(
-      base::MatchPattern(speech_monitor_.GetNextUtterance(), "about:blank*"));
 }
 
 IN_PROC_BROWSER_TEST_P(SpokenFeedbackTest, TypeInOmnibox) {
   EnableChromeVox();
 
-  chrome::ExecuteCommand(browser(), IDC_FOCUS_LOCATION);
-  EXPECT_EQ("Address and search bar", speech_monitor_.GetNextUtterance());
-  EXPECT_EQ("about:blank", speech_monitor_.GetNextUtterance());
-  EXPECT_EQ("Edit text", speech_monitor_.GetNextUtterance());
-  EXPECT_EQ("main", speech_monitor_.GetNextUtterance());
-  EXPECT_EQ("Tool bar", speech_monitor_.GetNextUtterance());
-  EXPECT_TRUE(
-      base::MatchPattern(speech_monitor_.GetNextUtterance(), "*about:blank*"));
-
+  // Location bar has focus by default so just start typing.
   SendKeyPress(ui::VKEY_X);
   EXPECT_EQ("x", speech_monitor_.GetNextUtterance());
 
   SendKeyPress(ui::VKEY_Y);
   EXPECT_EQ("y", speech_monitor_.GetNextUtterance());
 
   SendKeyPress(ui::VKEY_Z);
   EXPECT_EQ("z", speech_monitor_.GetNextUtterance());
 
   SendKeyPress(ui::VKEY_BACK);
@@ skipping 104 matching lines @@
       break;
   }
 
   SendKeyPress(ui::VKEY_TAB);
   while (true) {
     std::string utterance = speech_monitor_.GetNextUtterance();
     if (base::MatchPattern(utterance, "Button"))
       break;
   }
 
-  // Compat next element.
-  SendKeyPressWithSearchAndShift(ui::VKEY_RIGHT);
-  while (true) {
-    std::string utterance = speech_monitor_.GetNextUtterance();
-    if (base::MatchPattern(utterance, "Button"))
-      break;
-  }
+  // Next element.
+  SendKeyPressWithSearch(ui::VKEY_RIGHT);
+  EXPECT_TRUE(base::MatchPattern(speech_monitor_.GetNextUtterance(), "*"));
+  EXPECT_TRUE(base::MatchPattern(speech_monitor_.GetNextUtterance(), "Button"));
 
-  // Compat next button.
-  SendKeyPressWithSearchAndShift(ui::VKEY_N);
-  SendKeyPress(ui::VKEY_B);
-  while (true) {
-    std::string utterance = speech_monitor_.GetNextUtterance();
-    if (base::MatchPattern(utterance, "Button"))
-      break;
-  }
+  // Next button.
+  SendKeyPressWithSearch(ui::VKEY_B);
+  EXPECT_TRUE(base::MatchPattern(speech_monitor_.GetNextUtterance(), "*"));
+  EXPECT_TRUE(base::MatchPattern(speech_monitor_.GetNextUtterance(), "Button"));
 
   // Navigate to Bluetooth sub-menu and open it.
   while (true) {
     SendKeyPress(ui::VKEY_TAB);
     std::string content = speech_monitor_.GetNextUtterance();
     std::string role = speech_monitor_.GetNextUtterance();
     if (base::MatchPattern(content, "*Bluetooth*") &&
         base::MatchPattern(role, "Button"))
       break;
   }
@@ skipping 66 matching lines @@
   EXPECT_EQ("Button", speech_monitor_.GetNextUtterance());
 }
 
 #if defined(MEMORY_SANITIZER)
 // Fails under MemorySanitizer: http://crbug.com/472125
 #define MAYBE_ChromeVoxShiftSearch DISABLED_ChromeVoxShiftSearch
 #else
 #define MAYBE_ChromeVoxShiftSearch ChromeVoxShiftSearch
 #endif
 IN_PROC_BROWSER_TEST_P(SpokenFeedbackTest, MAYBE_ChromeVoxShiftSearch) {
-  LoadChromeVoxAndThenNavigateToURL(
+  EnableChromeVox();
+
+  ui_test_utils::NavigateToURL(
+      browser(),
       GURL("data:text/html;charset=utf-8,<button autofocus>Click me</button>"));
   while (true) {
     std::string utterance = speech_monitor_.GetNextUtterance();
-    if (utterance == "Click me")
-      break;
-  }
-  EXPECT_EQ("Button", speech_monitor_.GetNextUtterance());
-
-  // Press Search+Shift+/ to enter ChromeVox's "find in page".
-  SendKeyPressWithSearchAndShift(ui::VKEY_OEM_2);
-  EXPECT_EQ("Find in page.", speech_monitor_.GetNextUtterance());
-  EXPECT_EQ(",", speech_monitor_.GetNextUtterance());
-  EXPECT_EQ("Enter a search query.", speech_monitor_.GetNextUtterance());
-}
-
-#if defined(MEMORY_SANITIZER)
-// Fails under MemorySanitizer: http://crbug.com/472125
-#define MAYBE_ChromeVoxPrefixKey DISABLED_ChromeVoxPrefixKey
-#else
-#define MAYBE_ChromeVoxPrefixKey ChromeVoxPrefixKey
-#endif
-IN_PROC_BROWSER_TEST_P(SpokenFeedbackTest, MAYBE_ChromeVoxPrefixKey) {
-  LoadChromeVoxAndThenNavigateToURL(
-      GURL("data:text/html;charset=utf-8,<button autofocus>Click me</button>"));
-  while (true) {
-    std::string utterance = speech_monitor_.GetNextUtterance();
     if (utterance == "Click me")
       break;
   }
   EXPECT_EQ("Button", speech_monitor_.GetNextUtterance());
 
-  // Press the prefix key Ctrl+';' followed by '/'
-  // to enter ChromeVox's "find in page".
-  SendKeyPressWithControl(ui::VKEY_OEM_1);
-  SendKeyPress(ui::VKEY_OEM_2);
+  // Press Search+/ to enter ChromeVox's "find in page".
+  SendKeyPressWithSearch(ui::VKEY_OEM_2);
+  EXPECT_EQ(", window", speech_monitor_.GetNextUtterance());
+  EXPECT_EQ("webView", speech_monitor_.GetNextUtterance());
   EXPECT_EQ("Find in page.", speech_monitor_.GetNextUtterance());
-  EXPECT_EQ(",", speech_monitor_.GetNextUtterance());
-  EXPECT_EQ("Enter a search query.", speech_monitor_.GetNextUtterance());
 }
 
 #if defined(MEMORY_SANITIZER)
 // Fails under MemorySanitizer: http://crbug.com/472125
 #define MAYBE_ChromeVoxNavigateAndSelect DISABLED_ChromeVoxNavigateAndSelect
 #else
 #define MAYBE_ChromeVoxNavigateAndSelect ChromeVoxNavigateAndSelect
 #endif
 IN_PROC_BROWSER_TEST_P(SpokenFeedbackTest, MAYBE_ChromeVoxNavigateAndSelect) {
-  LoadChromeVoxAndThenNavigateToURL(
-      GURL("data:text/html;charset=utf-8,"
-           "<h1>Title</h1>"
-           "<button autofocus>Click me</button>"));
+  EnableChromeVox();
+
+  ui_test_utils::NavigateToURL(browser(),
+                               GURL("data:text/html;charset=utf-8,"
+                                    "<h1>Title</h1>"
+                                    "<button autofocus>Click me</button>"));
   while (true) {
     std::string utterance = speech_monitor_.GetNextUtterance();
     if (utterance == "Click me")
       break;
   }
   EXPECT_EQ("Button", speech_monitor_.GetNextUtterance());
 
-  // Press Search+Shift+Up to navigate to the previous item.
-  SendKeyPressWithSearchAndShift(ui::VKEY_UP);
+  // Press Search+Left to navigate to the previous item.
+  SendKeyPressWithSearch(ui::VKEY_LEFT);
   EXPECT_EQ("Title", speech_monitor_.GetNextUtterance());
   EXPECT_EQ("Heading 1", speech_monitor_.GetNextUtterance());
 
-  // Press Search+Shift+S to select the text.
-  SendKeyPressWithSearchAndShift(ui::VKEY_S);
-  EXPECT_EQ("Start selection", speech_monitor_.GetNextUtterance());
+  // Press Search+S to select the text.
+  SendKeyPressWithSearch(ui::VKEY_S);
   EXPECT_EQ("Title", speech_monitor_.GetNextUtterance());
-  EXPECT_EQ(", selected", speech_monitor_.GetNextUtterance());
+  EXPECT_EQ("selected", speech_monitor_.GetNextUtterance());
 
   // Press again to end the selection.
-  SendKeyPressWithSearchAndShift(ui::VKEY_S);
+  SendKeyPressWithSearch(ui::VKEY_S);
   EXPECT_EQ("End selection", speech_monitor_.GetNextUtterance());
   EXPECT_EQ("Title", speech_monitor_.GetNextUtterance());
 }
 
-// http://crbug.com/628060
-IN_PROC_BROWSER_TEST_P(SpokenFeedbackTest, DISABLED_ChromeVoxStickyMode) {
-  LoadChromeVoxAndThenNavigateToURL(
-      GURL("data:text/html;charset=utf-8,"
-           "<label>Enter your name <input autofocus></label>"
-           "<p>One</p>"
-           "<h2>Two</h2>"));
-  while (speech_monitor_.GetNextUtterance() != "Enter your name") {
-  }
-  EXPECT_EQ("Edit text", speech_monitor_.GetNextUtterance());
+IN_PROC_BROWSER_TEST_P(SpokenFeedbackTest, ChromeVoxNextStickyMode) {
+  EnableChromeVox();
 
-  // Press the sticky-key sequence: Search Search.
-  SendKeyPress(ui::VKEY_LWIN);
-
-  // Sticky key has a minimum 100 ms check to prevent key repeat from toggling
-  // it.
-  content::BrowserThread::PostDelayedTask(
-      content::BrowserThread::UI, FROM_HERE,
-      base::Bind(&LoggedInSpokenFeedbackTest::SendKeyPress,
-                 base::Unretained(this), ui::VKEY_LWIN),
-      base::TimeDelta::FromMilliseconds(200));
-
-  EXPECT_EQ("Sticky mode enabled", speech_monitor_.GetNextUtterance());
-
-  // Even once we hear "sticky mode enabled" from the ChromeVox background
-  // page, there's a short window of time when the content script still
-  // hasn't switched to sticky mode. That's why we're focused on a text box.
-  // Keep pressing the '/' key. If sticky mode is off, it will echo the word
-  // "slash". If sticky mode is on, it will open "Find in page". Keep pressing
-  // '/' until we get "Find in page.".
-  PressRepeatedlyUntilUtterance(ui::VKEY_OEM_2, "Find in page.");
-  while (speech_monitor_.GetNextUtterance() != "Enter a search query.") {
-  }
-
-  // Press Esc to exit Find in Page mode.
-  SendKeyPress(ui::VKEY_ESCAPE);
-  EXPECT_EQ("Exited", speech_monitor_.GetNextUtterance());
-  while (speech_monitor_.GetNextUtterance() != "Find in page.") {
-  }
-}
-
-IN_PROC_BROWSER_TEST_P(SpokenFeedbackTest, ChromeVoxNextStickyMode) {
-  LoadChromeVoxAndThenNavigateToURL(
-      GURL("data:text/html;charset=utf-8,<button autofocus>Click me</button>"
-           "<!-- chromevox_next_test -->"));
+  ui_test_utils::NavigateToURL(
+      browser(),
+      GURL("data:text/html;charset=utf-8,<button autofocus>Click me</button>"));
   while ("Button" != speech_monitor_.GetNextUtterance()) {
   }
 
   // Press the sticky-key sequence: Search Search.
   SendKeyPress(ui::VKEY_LWIN);
 
   // Sticky key has a minimum 100 ms check to prevent key repeat from toggling
   // it.
   content::BrowserThread::PostDelayedTask(
       content::BrowserThread::UI, FROM_HERE,
@@ skipping 58 matching lines @@
   DISALLOW_COPY_AND_ASSIGN(GuestSpokenFeedbackTest);
 };
 
 IN_PROC_BROWSER_TEST_F(GuestSpokenFeedbackTest, FocusToolbar) {
   EnableChromeVox();
 
   chrome::ExecuteCommand(browser(), IDC_FOCUS_TOOLBAR);
 
   EXPECT_EQ("Reload", speech_monitor_.GetNextUtterance());
   EXPECT_EQ("Button", speech_monitor_.GetNextUtterance());
-  EXPECT_EQ("main", speech_monitor_.GetNextUtterance());
-  EXPECT_EQ("Tool bar", speech_monitor_.GetNextUtterance());
-  EXPECT_TRUE(
-      base::MatchPattern(speech_monitor_.GetNextUtterance(), "about:blank*"));
 }
 
 //
 // Spoken feedback tests of the out-of-box experience.
 //
 
 class OobeSpokenFeedbackTest : public LoginManagerTest {
  protected:
   OobeSpokenFeedbackTest() : LoginManagerTest(false) {}
   ~OobeSpokenFeedbackTest() override {}
@@ skipping 42 matching lines @@
   ASSERT_TRUE(ui_test_utils::SendKeyPressToWindowSync(
       window, ui::VKEY_TAB, false, true /*shift*/, false, false));
   while (speech_monitor_.GetNextUtterance() != "Select your language:") {
   }
   EXPECT_EQ("English ( United States)", speech_monitor_.GetNextUtterance());
   EXPECT_TRUE(base::MatchPattern(speech_monitor_.GetNextUtterance(),
                                  "Combo box * of *"));
 }
 
 }  // namespace chromeos