Chromium Code Reviews Chromium Code Reviews
Issue 234933002:
Dispatch change event for Input type=range for touchEnd (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| Index: LayoutTests/fast/forms/range/slider-onchange-event.html |
| diff --git a/LayoutTests/fast/forms/range/slider-onchange-event.html b/LayoutTests/fast/forms/range/slider-onchange-event.html |
| index 3dc3ca1016fc865942c39b7dd589b57e4136104e..4d54e83e95d81db6661daf96a558bc75d44403c9 100644 |
| --- a/LayoutTests/fast/forms/range/slider-onchange-event.html |
| +++ b/LayoutTests/fast/forms/range/slider-onchange-event.html |
| @@ -24,13 +24,24 @@ |
| eventSender.mouseMoveTo(slider.offsetLeft + 7, slider.offsetTop + 7); |
| eventSender.mouseDown(); |
| eventSender.mouseUp(); |
| + // On touch events, check if change event is dispatched at touch end |
| + var x = slider.offsetLeft; |
| + var y = slider.offsetTop + slider.clientHeight/2; |
|
tkent
2014/04/14 00:35:47
nit: needs spaces around '/'
|
| + var w = slider.clientWidth; |
| + |
| + eventSender.clearTouchPoints(); |
| + eventSender.addTouchPoint(x + w/2, y); |
|
tkent
2014/04/14 00:35:47
Ditto.
|
| + eventSender.touchStart(); |
| + eventSender.touchMove(); |
| + eventSender.releaseTouchPoint(0); |
| + eventSender.touchEnd(); |
| } |
| } |
| </script> |
| </head> |
| <body onload="test()"> |
| This tests that slider controls get the onchange event and that it is sent after the DOMInput event.<br> |
| - <input type="range" id="slider" onchange="log('change')" oninput="log('input')" value=0> |
| + <input type="range" id="slider" onchange="log('change')" oninput="log('input')" value=0 width=200px height=300px> |
| <div id="res"></div> |
| </body> |
| </html> |
