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

Unified Diff: chrome/test/data/webui/menu_test.html

Issue 1358893003: Ignore mouseup shortly after showing cr.ui.Menu (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: . Created 5 years, 3 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/menu_test.html
diff --git a/chrome/test/data/webui/menu_test.html b/chrome/test/data/webui/menu_test.html
index 7e0e9205a06d1766491e64abf1a9c0e387aebfd6..83b188fcbdce611a62d4816a20f210340fdc9942 100644
--- a/chrome/test/data/webui/menu_test.html
+++ b/chrome/test/data/webui/menu_test.html
@@ -18,6 +18,36 @@ function testHandleMouseOver() {
assertTrue(called);
}
+function testHandleMouseUp() {
+ var realNow = Date.now;
+ Date.now = function() { return 10; };
+
+ var menu = new cr.ui.Menu;
+ menu.show({x: 5, y: 5});
+
+ function mouseUpAt(x, y) {
+ return menu.dispatchEvent(new MouseEvent('mouseup', {
+ bubbles: true,
+ cancelable: true,
+ target: menu,
+ screenX: x,
+ screenY: y,
+ }));
+ }
+
+ // Stop mouseups at the same time and position.
+ assertFalse(mouseUpAt(5, 5));
+
+ // Allow mouseups with different positions but the same time.
+ assertTrue(mouseUpAt(50, 50));
+
+ // Alow mouseups with the same position but different times.
+ Date.now = function() { return 1000; };
+ assertTrue(mouseUpAt(5, 5));
+
+ Date.now = realNow;
+}
+
</script>
</body>
</html>
« no previous file with comments | « no previous file | ui/webui/resources/js/cr/ui/context_menu_handler.js » ('j') | ui/webui/resources/js/cr/ui/menu.js » ('J')

Powered by Google App Engine
This is Rietveld 408576698