Bug 32403

Summary: Chromium: select reverts back if the value is selected via keyboard when popup is visible
Product: WebKit Reporter: Victor Wang <victorw>
Component: Layout and RenderingAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, dglazkov, fishd, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: All   
Attachments:
Description Flags
Proposed patch none

Victor Wang
Reported 2009-12-10 18:05:42 PST
Here is the step to repro it: 1. Open a webpage that has select element. For example, http://www.wikipedia.org, the language list. 2. Click the select to open the popup list. 3. Use keyboard to select a value by pressing the first letter of the value. 4. Click the select again to close the popup list, it should stay at the value from step 2. 5. Click anywhere else on the page to move the focus out of the select. In this instance, the item reverts back to the selection from before step 2. This happens to chromium. The issue does not apply to Safari mac version because the popup list covers the menulist. Safari windows version works fine.
Attachments
Proposed patch (3.01 KB, patch)
2009-12-10 18:34 PST, Victor Wang 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Victor Wang
Comment 1 2009-12-10 18:34:23 PST
Created attachment 44657 [details] Proposed patch
WebKit Review Bot
Comment 2 2009-12-10 18:36:47 PST
style-queue ran check-webkit-style on attachment 44657 [details] without any errors.
Darin Fisher (:fishd, Google)
Comment 3 2009-12-11 09:53:53 PST
I think we really need to make it possible to exercise popup menus from TestShell.
Darin Fisher (:fishd, Google)
Comment 4 2009-12-11 09:54:41 PST
(whoops, i should wait for the patch to land before resolving this bug.)
WebKit Commit Bot
Comment 5 2009-12-11 10:03:54 PST
Comment on attachment 44657 [details] Proposed patch Clearing flags on attachment: 44657 Committed r52001: <http://trac.webkit.org/changeset/52001>
WebKit Commit Bot
Comment 6 2009-12-11 10:04:04 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.