RESOLVED FIXED 74384
[Non-Mac] Change event should be fired when changing option by using keyboard.
https://bugs.webkit.org/show_bug.cgi?id=74384
Summary [Non-Mac] Change event should be fired when changing option by using keyboard.
yosin
Reported 2011-12-13 00:23:18 PST
onchange event should be fired when changing option by using keyboard. This bug is imported from http://crbug.com/90447
Attachments
Proposed patch (5.02 KB, patch)
2011-12-14 00:11 PST, Rakesh
no flags
Proposed patch (6.57 KB, patch)
2011-12-14 01:54 PST, Rakesh
no flags
Updated patch (6.63 KB, patch)
2011-12-14 03:19 PST, Rakesh
no flags
Rakesh
Comment 1 2011-12-14 00:11:32 PST
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 2 2011-12-14 00:22:22 PST
Comment on attachment 119169 [details] Proposed patch The code looks good. Would you move the test to fast/forms/select/ please?
Kent Tamura
Comment 3 2011-12-14 00:30:36 PST
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
Comment 4 2011-12-14 01:15:21 PST
(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
Comment 5 2011-12-14 01:54:35 PST
Created attachment 119178 [details] Proposed patch Skipped test for MAC, moved the test to fast/forms/select/
Kent Tamura
Comment 6 2011-12-14 01:56:13 PST
Comment on attachment 119178 [details] Proposed patch Looks good.
WebKit Review Bot
Comment 7 2011-12-14 02:05:14 PST
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
Comment 8 2011-12-14 03:19:39 PST
Created attachment 119190 [details] Updated patch Changes on latest code.
WebKit Review Bot
Comment 9 2011-12-14 05:00:37 PST
Comment on attachment 119190 [details] Updated patch Clearing flags on attachment: 119190 Committed r102767: <http://trac.webkit.org/changeset/102767>
WebKit Review Bot
Comment 10 2011-12-14 05:00:41 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.