Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(729)

Issue 53017: Fix the form autofill showing when pressing the up arrow key (Closed)

Created:
11 years, 9 months ago by jcampan
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

When in a form text field, pressing the up/down arrow keys should bring up the autofill popup. Pressing up would not trigger that behavior. This happened because the default processing of the up arrow key is to move the caret to the beginning of the text input. Showing the autofill is done as a posted task. Because the caret was not at the end of the text-field by the time that task got executed, the popup would not show. The fix is to not default process the up/down arrow key messages when an autofill popup will be shown. Also fixed the WebViewImpl::HideAutoCompletePopup (harmless bug, calling it twice would cause it to hide the already hidden popup). BUG=6437 TEST=Make sure up/down arrow keys trigger the autofill menu correctly (especially when there is already text in the text-field). Make sure autofill still works as expected. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=12428

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+22 lines, -10 lines) Patch
M webkit/glue/editor_client_impl.h View 2 chunks +6 lines, -2 lines 0 comments Download
M webkit/glue/editor_client_impl.cc View 4 chunks +15 lines, -7 lines 0 comments Download
M webkit/glue/webview_impl.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
jcampan
11 years, 9 months ago (2009-03-24 19:33:10 UTC) #1
tim (not reviewing)
11 years, 9 months ago (2009-03-25 00:02:17 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698