Revert "Chrome OS: Fix and re-enable OobeLocalizationTest"

This reverts commit 6ce17133.

Reason for revert: Failing on linux-chromeos-rel
https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/linux-chromeos-rel/16712

Original change's description:
> Chrome OS: Fix and re-enable OobeLocalizationTest
> 
> Bug: 633246
> Change-Id: Id6c302696c955401a991c0235d86f05d03019036
> Reviewed-on: https://chromium-review.googlesource.com/c/1355045
> Reviewed-by: Jacob Dufault <jdufault@chromium.org>
> Commit-Queue: Alexander Alekseev <alemate@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#612867}

TBR=alemate@chromium.org,jdufault@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 633246
Change-Id: I8d8994bc4d89fd87427d055a2946e6168a4d2f89
Reviewed-on: https://chromium-review.googlesource.com/c/1358106Reviewed-by: calamity <calamity@chromium.org>
Commit-Queue: calamity <calamity@chromium.org>
Cr-Commit-Position: refs/heads/master@{#613001}

chrome/browser/chromeos/login/oobe_localization_browsertest.cc

@@ -47,13 +47,8 @@ namespace {
 const int kTimeoutSeconds = 2;
 
 // OOBE constants.
-const char kLanguageSelect[] = "languageSelect";
-const char kKeyboardSelect[] = "keyboardSelect";
-
-std::string GetGetSelectStatement(const std::string& selectId) {
-  return "document.getElementById('oobe-welcome-md').$." + selectId +
-         ".$.select";
-}
+const char kLocaleSelect[] = "language-select";
+const char kKeyboardSelect[] = "keyboard-select";
 
 const char kUSLayout[] = "xkb:us::eng";
 class TimedRunLoop {
@@ -172,21 +167,23 @@ struct LocalizationTestParams {
 
     {"ru,de", "xkb:ru::rus", "ru,de", kUSLayout, "xkb:us::eng"},
 
+    // TODO(alemate/michaelpg): Figure out why these tests are failing
+    // and re-enable them. crbug.com/422702.
     // ------------------ Regional Locales
     // Syntetic example to test correct merging of different locales.
-    {"fr-CH,it-CH,de-CH",
-     "xkb:fr::fra,xkb:it::ita,xkb:de::ger",
-     "fr-CH,it-CH,de-CH",
-     "xkb:fr::fra",
-     "xkb:fr::fra,xkb:it::ita,xkb:de::ger,[xkb:be::fra,xkb:ca::fra,"
-         "xkb:ch:fr:fra,xkb:ca:multix:fra,xkb:us::eng]"},
+    // {"fr-CH,it-CH,de-CH",
+    //  "xkb:fr::fra,xkb:it::ita,xkb:de::ger",
+    //  "fr-CH,it-CH,de-CH",
+    //  "xkb:fr::fra",
+    //  "xkb:fr::fra,xkb:it::ita,xkb:de::ger,[xkb:be::fra,xkb:ca::fra,"
+    //      "xkb:ch:fr:fra,xkb:ca:multix:fra,xkb:us::eng]"},
 
     // Another syntetic example. Check that british keyboard is available.
-    {"en-AU",
-     "xkb:us::eng",
-     "en-AU",
-     "xkb:us::eng",
-     "xkb:us::eng,[xkb:gb:extd:eng,xkb:gb:dvorak:eng]"},
+    // {"en-AU",
+    //  "xkb:us::eng",
+    //  "en-AU",
+    //  "xkb:us::eng",
+    //  "xkb:us::eng,[xkb:gb:extd:eng,xkb:gb:dvorak:eng]"},
 };
 
 class OobeLocalizationTest
@@ -250,11 +247,11 @@ bool OobeLocalizationTest::VerifyInitialOptions(const char* select_id,
                                                 bool check_separator) {
   const std::string expression = base::StringPrintf(
       "(function () {\n"
-      "  let select = %s;\n"
+      "  var select = document.querySelector('#%s');\n"
       "  if (!select)\n"
       "    return false;\n"
-      "  let values = '%s'.split(',');\n"
-      "  let correct = select.selectedIndex == 0;\n"
+      "  var values = '%s'.split(',');\n"
+      "  var correct = select.selectedIndex == 0;\n"
       "  for (var i = 0; i < values.length && correct; i++) {\n"
       "    if (select.options[i].value != values[i])\n"
       "      correct = false;\n"
@@ -263,7 +260,7 @@ bool OobeLocalizationTest::VerifyInitialOptions(const char* select_id,
       "    correct = select.children[values.length].tagName === 'OPTGROUP';\n"
       "  return correct;\n"
       "})()",
-      GetGetSelectStatement(select_id).c_str(), values, check_separator);
+      select_id, values, check_separator);
   const bool execute_status = checker.GetBool(expression);
   EXPECT_TRUE(execute_status) << expression;
   return execute_status;
@@ -273,7 +270,7 @@ bool OobeLocalizationTest::VerifyOptionExists(const char* select_id,
                                               const char* value) {
   const std::string expression = base::StringPrintf(
       "(function () {\n"
-      "  var select = %s;\n"
+      "  var select = document.querySelector('#%s');\n"
       "  if (!select)\n"
       "    return false;\n"
       "  for (var i = 0; i < select.options.length; i++) {\n"
@@ -282,7 +279,7 @@ bool OobeLocalizationTest::VerifyOptionExists(const char* select_id,
       "  }\n"
       "  return false;\n"
       "})()",
-      GetGetSelectStatement(select_id).c_str(), value);
+      select_id, value);
   const bool execute_status = checker.GetBool(expression);
   EXPECT_TRUE(execute_status) << expression;
   return execute_status;
@@ -290,11 +287,13 @@ bool OobeLocalizationTest::VerifyOptionExists(const char* select_id,
 
 std::string OobeLocalizationTest::DumpOptions(const char* select_id) {
   const std::string expression = base::StringPrintf(
+      "\n"
       "(function () {\n"
-      "  var select = %s;\n"
+      "  var selector = '#%s';\n"
       "  var divider = ',';\n"
+      "  var select = document.querySelector(selector);\n"
       "  if (!select)\n"
-      "    return 'select statement for \"%s\" failed.';\n"
+      "    return 'document.querySelector(' + selector + ') failed.';\n"
       "  var dumpOptgroup = function(group) {\n"
       "    var result = '';\n"
       "    for (var i = 0; i < group.children.length; i++) {\n"
@@ -330,7 +329,7 @@ std::string OobeLocalizationTest::DumpOptions(const char* select_id) {
       "  }\n"
       "  return result;\n"
       "})()\n",
-      GetGetSelectStatement(select_id).c_str(), select_id);
+      select_id);
   return checker.GetString(expression);
 }
 
@@ -367,11 +366,10 @@ void OobeLocalizationTest::RunLocalizationTest() {
   const std::string first_language =
       expected_locale.substr(0, expected_locale.find(','));
   bool done = false;
-  const std::string get_select_statement =
-      GetGetSelectStatement(kLanguageSelect);
   const std::string waiting_script = base::StringPrintf(
+      "var screenElement = document.getElementById('language-select');"
       "function SendReplyIfAcceptEnabled() {"
-      "  if (%s.value != '%s')"
+      "  if ($('language-select').value != '%s')"
       "    return false;"
       "  domAutomationController.send(true);"
       "  observer.disconnect();"
@@ -379,11 +377,10 @@ void OobeLocalizationTest::RunLocalizationTest() {
       "}"
       "var observer = new MutationObserver(SendReplyIfAcceptEnabled);"
       "if (!SendReplyIfAcceptEnabled()) {"
-      "  let options = { attributes: true };"
-      "  observer.observe(%s, options);"
+      "  var options = { attributes: true };"
+      "  observer.observe(screenElement, options);"
       "}",
-      get_select_statement.c_str(), first_language.c_str(),
-      get_select_statement.c_str());
+      first_language.c_str());
 
   ASSERT_TRUE(content::ExecuteScriptAndExtractBool(
       LoginDisplayHost::default_host()->GetOobeUI()->web_ui()->GetWebContents(),
@@ -394,17 +391,15 @@ void OobeLocalizationTest::RunLocalizationTest() {
                                ->web_ui()
                                ->GetWebContents());
 
-  if (!VerifyInitialOptions(kLanguageSelect, expected_locale.c_str(), true)) {
-    LOG(ERROR) << "Actual value of " << kLanguageSelect << ":\n"
-               << DumpOptions(kLanguageSelect);
-    EXPECT_TRUE(false);
+  if (!VerifyInitialOptions(kLocaleSelect, expected_locale.c_str(), true)) {
+    LOG(ERROR) << "Actual value of " << kLocaleSelect << ":\n"
+               << DumpOptions(kLocaleSelect);
   }
   if (!VerifyInitialOptions(
           kKeyboardSelect,
           TranslateXKB2Extension(expected_keyboard_layout).c_str(), false)) {
     LOG(ERROR) << "Actual value of " << kKeyboardSelect << ":\n"
                << DumpOptions(kKeyboardSelect);
-    EXPECT_TRUE(false);
   }
 
   // Make sure we have a fallback keyboard.
@@ -413,7 +408,6 @@ void OobeLocalizationTest::RunLocalizationTest() {
           extension_ime_util::GetInputMethodIDByEngineID(kUSLayout).c_str())) {
     LOG(ERROR) << "Actual value of " << kKeyboardSelect << ":\n"
                << DumpOptions(kKeyboardSelect);
-    EXPECT_TRUE(false);
   }
 
   // Note, that sort order is locale-specific, but is unlikely to change.
@@ -429,7 +423,7 @@ void OobeLocalizationTest::RunLocalizationTest() {
       language::prefs::kApplicationLocale, std::string());
 }
 
-IN_PROC_BROWSER_TEST_P(OobeLocalizationTest, LocalizationTest) {
+IN_PROC_BROWSER_TEST_P(OobeLocalizationTest, DISABLED_LocalizationTest) {
   RunLocalizationTest();
 }