Chromium Code Reviews Chromium Code Reviews
Issue 128133002:
Update of change event for input type number (Closed)
Base URL: https://chromium.googlesource.com/chromium/blink.git@master| Index: LayoutTests/fast/forms/number/number-spinbutton-changeevent-trigger.html |
| diff --git a/LayoutTests/fast/forms/number/number-spinbutton-changeevent-trigger.html b/LayoutTests/fast/forms/number/number-spinbutton-changeevent-trigger.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..2e5c2511512ebfce364fa0d2230b5cf5f885a942 |
| --- /dev/null |
| +++ b/LayoutTests/fast/forms/number/number-spinbutton-changeevent-trigger.html |
| @@ -0,0 +1,33 @@ |
| +<!DOCTYPE> |
| +<html> |
| +<body> |
| +<script src="../../../resources/js-test.js"></script> |
| +<input type="number" id="num1" onchange="handleChangeEvent(this);" oninput="handleInputEvent(this);"> |
| +<script> |
| +var changeEventCounter = 0; |
| +var inputEventCounter = 0; |
| + |
| +function handleChangeEvent(element){ |
| + ++changeEventCounter; |
|
tkent
2014/01/17 02:19:20
We usually use four-space indentation in JavaScrip
|
| +} |
| + |
| +function handleInputEvent(element){ |
| + ++inputEventCounter; |
| +} |
| + |
| +var numberInput1= document.getElementById('num1'); |
| +numberInput1.focus(); |
| +eventSender.mouseMoveTo(0, 0); |
| +eventSender.mouseMoveTo(numberInput1.offsetLeft + numberInput1.offsetWidth - 10, numberInput1.offsetTop + numberInput1.offsetHeight / 4); |
| +eventSender.mouseDown(); |
| +debug('Only input event occurs on mouse down'); |
| +shouldBe('inputEventCounter', '1'); |
| +shouldBe('changeEventCounter', '0'); |
|
tkent
2014/01/17 02:19:20
Good. This check demonstrates that change events
|
| +debug('Only change event occurs on mouse up'); |
| +eventSender.mouseUp(); |
| +shouldBe('inputEventCounter', '1'); |
| +shouldBe('changeEventCounter', '1'); |
| + |
| +</script> |
| +</body> |
| +</html> |
