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

Unified Diff: chrome/test/data/webui/md_history/history_overflow_menu_test.js

Issue 2104013004: MD WebUI: Reimplement cr-shared-menu using iron-dropdown. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Nits Created 4 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: chrome/test/data/webui/md_history/history_overflow_menu_test.js
diff --git a/chrome/test/data/webui/md_history/history_overflow_menu_test.js b/chrome/test/data/webui/md_history/history_overflow_menu_test.js
index 01836a7f3545f911d201889063cd9a60e240be06..b7ca2eebe61bad2ebc20bee02a3336abf227fce1 100644
--- a/chrome/test/data/webui/md_history/history_overflow_menu_test.js
+++ b/chrome/test/data/webui/md_history/history_overflow_menu_test.js
@@ -61,15 +61,6 @@ cr.define('md_history.history_overflow_menu_test', function() {
element.$.sharedMenu.lastAnchor_);
});
- test('keyboard input for closing menu', function() {
- // Test that pressing escape on the document closes the menu. In the
- // actual page, it will take two presses to close the menu due to focus.
- // TODO(yingran): Fix this behavior to only require one key press.
- element.toggleMenu_(MENU_EVENT);
- MockInteractions.pressAndReleaseKeyOn(document.body, 27);
- assertEquals(false, element.$.sharedMenu.menuOpen);
- });
-
teardown(function() {
element.$.sharedMenu.lastAnchor_ = null;
});

Powered by Google App Engine
This is Rietveld 408576698