Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(54)

Side by Side Diff: chrome/browser/ui/webui/options/language_options_interactive_uitest.cc

Issue 489983002: Disable LanguageOptionsWebUITest.TestListTabAccessibility. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/prefs/pref_service.h" 5 #include "base/prefs/pref_service.h"
6 #include "chrome/browser/ui/browser_window.h" 6 #include "chrome/browser/ui/browser_window.h"
7 #include "chrome/browser/ui/chrome_pages.h" 7 #include "chrome/browser/ui/chrome_pages.h"
8 #include "chrome/browser/ui/tabs/tab_strip_model.h" 8 #include "chrome/browser/ui/tabs/tab_strip_model.h"
9 #include "chrome/common/pref_names.h" 9 #include "chrome/common/pref_names.h"
10 #include "chrome/common/url_constants.h" 10 #include "chrome/common/url_constants.h"
(...skipping 97 matching lines...) Expand 10 before | Expand all | Expand 10 after
108 active_frame, 108 active_frame,
109 get_children_of_current_element_script, 109 get_children_of_current_element_script,
110 &languages)); 110 &languages));
111 EXPECT_EQ("English (United States)SpanishFrench", languages); 111 EXPECT_EQ("English (United States)SpanishFrench", languages);
112 } 112 }
113 113
114 // This test will validate that the language webui is accessible through 114 // This test will validate that the language webui is accessible through
115 // the keyboard. 115 // the keyboard.
116 // This test must be updated if the tab order of the elements on this page 116 // This test must be updated if the tab order of the elements on this page
117 // is chagned. 117 // is chagned.
118 IN_PROC_BROWSER_TEST_F(LanguageOptionsWebUITest, TestListTabAccessibility) { 118 // flaky: http://crbug.com/405711
119 IN_PROC_BROWSER_TEST_F(LanguageOptionsWebUITest,
120 DISABLED_TestListTabAccessibility) {
119 // Verify that the language list is focused by default. 121 // Verify that the language list is focused by default.
120 std::string original_id = GetActiveElementId(); 122 std::string original_id = GetActiveElementId();
121 EXPECT_EQ("language-options-list", original_id); 123 EXPECT_EQ("language-options-list", original_id);
122 124
123 // Press tab to select the next element. 125 // Press tab to select the next element.
124 ASSERT_TRUE(PressKey(ui::VKEY_TAB)); 126 ASSERT_TRUE(PressKey(ui::VKEY_TAB));
125 127
126 // Make sure that the element is now the button that is next in the tab order. 128 // Make sure that the element is now the button that is next in the tab order.
127 // Checking that the list is no longer selected is not sufficient to validate 129 // Checking that the list is no longer selected is not sufficient to validate
128 // this use case because this test should fail if an item inside the list is 130 // this use case because this test should fail if an item inside the list is
129 // selected. 131 // selected.
130 std::string new_id = GetActiveElementId(); 132 std::string new_id = GetActiveElementId();
131 EXPECT_EQ("language-options-add-button", new_id); 133 EXPECT_EQ("language-options-add-button", new_id);
132 } 134 }
133 135
134 } // namespace language_options_ui_test 136 } // namespace language_options_ui_test
135 137
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698