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

Unified Diff: LayoutTests/fast/events/click-over-descendant-elements.html

Issue 110173005: Don't dispatch a 'click' event for mousedown/mouseup across form controls. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: fix a test Created 7 years 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: LayoutTests/fast/events/click-over-descendant-elements.html
diff --git a/LayoutTests/fast/events/click-over-descendant-elements.html b/LayoutTests/fast/events/click-over-descendant-elements.html
index 013dc52ecd357e754f129bfb383750e610a02ab8..ac921069d127049b25ba8430369b443bf2acd227 100644
--- a/LayoutTests/fast/events/click-over-descendant-elements.html
+++ b/LayoutTests/fast/events/click-over-descendant-elements.html
@@ -126,6 +126,14 @@ eventSender.mouseUp();
shouldBe('lastClickTarget', 'input1');
lastClickTarget = null;
+debug('Mousedown on a form control, and mouseup on an element outside:');
+eventSender.mouseMoveTo(inputRect.left + 8, (inputRect.top + inputRect.bottom) / 2);
+eventSender.mouseDown();
+eventSender.mouseMoveTo((spanRect.left + spanRect.right) / 2, spanRect.top + 1);
+eventSender.mouseUp();
+shouldBeNull('lastClickTarget');
+lastClickTarget = null;
+
container.remove();
button1.remove();
input1.remove();

Powered by Google App Engine
This is Rietveld 408576698