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

Unified Diff: chrome/browser/ui/webui/options/options_browsertest.js

Issue 2468453002: Disable failure test OptionsWebUITest.EnterPreventsDefault (Closed)
Patch Set: Created 4 years, 1 month 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/webui/options/options_browsertest.js
diff --git a/chrome/browser/ui/webui/options/options_browsertest.js b/chrome/browser/ui/webui/options/options_browsertest.js
index c80a4efced864f296f90421df4f84e700a361dcf..bbb019fea0d1f2675009f26c54794cacd111358d 100644
--- a/chrome/browser/ui/webui/options/options_browsertest.js
+++ b/chrome/browser/ui/webui/options/options_browsertest.js
@@ -311,11 +311,12 @@ TEST_F('OptionsWebUITest', 'EnableAndDisableDoNotTrack', function() {
dntCheckbox.click();
});
+// Fails on chromeos, http://crbug.com/660867
// Verify that preventDefault() is called on 'Enter' keydown events that trigger
// the default button. If this doesn't happen, other elements that may get
// focus (by the overlay closing for instance), will execute in addition to the
// default button. See crbug.com/268336.
-TEST_F('OptionsWebUITest', 'EnterPreventsDefault', function() {
+TEST_F('OptionsWebUITest', 'DISABLED_EnterPreventsDefault', function() {
var page = HomePageOverlay.getInstance();
PageManager.showPageByName(page.name);
var event = new KeyboardEvent('keydown', {
« 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