Bug 74384 – [Non-Mac] Change event should be fired when changing option by using keyboard.

• Component: Forms
• Status: RESOLVED FIXED

onchange event should be fired when changing option by using keyboard.

This bug is imported from
http://crbug.com/90447

[Rakesh]

Created attachment 119169 [details]
Proposed patch

For the up/down/home/end/pageup/pagedown key events, the DispatchChangeEvent option flag was not set

[Kent Tamura]

Comment on attachment 119169 [details]
Proposed patch

The code looks good.
Would you move the test to fast/forms/select/ please?

[Kent Tamura]

Comment on attachment 119169 [details]
Proposed patch

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

> LayoutTests/ChangeLog:11
> +        * fast/forms/menulist-onchange-fired-with-key-up-down-expected.txt: Added.
> +        * fast/forms/menulist-onchange-fired-with-key-up-down.html: Added.

The test won't work on Apple Mac and Chromium Mac.  We should add this test to LayoutTests/platform/mac/Skipped and LayoutTests/platform/chromium/test_expectations.txt.

[Rakesh]

(In reply to comment #2)
> (From update of attachment 119169 [details])
> The code looks good.
> Would you move the test to fast/forms/select/ please?

ok, will do that.

> > +        * fast/forms/menulist-onchange-fired-with-key-up-down-expected.txt: Added.
> > +        * fast/forms/menulist-onchange-fired-with-key-up-down.html: Added.
> 
> The test won't work on Apple Mac and Chromium Mac.  We should add this test to LayoutTests/platform/mac/Skipped and LayoutTests/platform/chromium/test_expectations.txt.

ok, will add these too.

Thanks for taking time to review.

[Rakesh]

Created attachment 119178 [details]
Proposed patch

Skipped test for MAC, moved the test to fast/forms/select/

[Kent Tamura]

Comment on attachment 119178 [details]
Proposed patch

Looks good.

[WebKit Review Bot]

Comment on attachment 119178 [details]
Proposed patch

Rejecting attachment 119178 [details] from commit-queue.

Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2

Last 500 characters of output:
-key-up-down.html
patching file LayoutTests/platform/chromium/test_expectations.txt
Hunk #1 FAILED at 3784.
1 out of 1 hunk FAILED -- saving rejects to file LayoutTests/platform/chromium/test_expectations.txt.rej
patching file LayoutTests/platform/mac/Skipped
Hunk #1 FAILED at 498.
1 out of 1 hunk FAILED -- saving rejects to file LayoutTests/platform/mac/Skipped.rej

Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--reviewer', u'Kent Tamura', u'--force']" exit_code: 1

Full output: http://queues.webkit.org/results/10873153

[Rakesh]

Created attachment 119190 [details]
Updated patch

Changes on latest code.

[WebKit Review Bot]

Comment on attachment 119190 [details]
Updated patch

Clearing flags on attachment: 119190

Committed r102767: <http://trac.webkit.org/changeset/102767>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.