Bug 107429 – INPUT_MULTIPLE_FIELDS_UI: should not dispatch 'input' events if the element value is not updated

• Status: RESOLVED
• Resolution: FIXED
• Component: Forms

[Kent Tamura]

INPUT_MULTIPLE_FIELDS_UI: should not dispatch 'input' events if the element value is not updated

[Kent Tamura]

Created attachment 183737 [details]
Patch

[Kentaro Hara]

Comment on attachment 183737 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=183737&action=review

> Source/WebCore/ChangeLog:20
> +        However we shoudl recalculate validity and call notifyFormStateChanged

Typo: should

> LayoutTests/fast/forms/time-multiple-fields/time-multiple-fields-keyboard-events.html:61
> +keyDown('7'); // -> 07:[--] --
> +keyDown('5'); // -> 07:[05] --
> +keyDown('6'); // -> 07:56 [--]
> +keyDown('A'); // -> 07:56 [AM]
>  shouldBeEqualToString('input.value', '07:56');
> +shouldBe('eventsCounter.input', '1');
> +shouldBe('eventsCounter.change', '1');

Sorry, would you elaborate on why the result becomes 1? (When are the events triggered?)

How about changing the test as follows to make it clear when the events are triggered?

eventCounters = {};
keyDown('7');
keyDown('5');
keyDown('6');
shouldBe('eventsCounter.input', '0');
shouldBe('eventsCounter.change', '0');
keyDown('A');
shouldBe('eventsCounter.input', '1');
shouldBe('eventsCounter.change', '1');

> LayoutTests/fast/forms/time-multiple-fields/time-multiple-fields-keyboard-events.html:122
> +shouldBe('eventsCounter.input', '1');
> +shouldBe('eventsCounter.change', '1');

Ditto.

[Kent Tamura]

Created attachment 183741 [details]
Patch 2

[Kentaro Hara]

Comment on attachment 183741 [details]
Patch 2

thanks

[WebKit Review Bot]

Comment on attachment 183741 [details]
Patch 2

Clearing flags on attachment: 183741

Committed r140324: <http://trac.webkit.org/changeset/140324>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.