diff options
| author | xji@chromium.org <xji@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-23 16:36:29 +0000 |
|---|---|---|
| committer | xji@chromium.org <xji@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-09-23 16:36:29 +0000 |
| commit | d2b0ab05ce248d278b8ce4124402c631c3ec921d (patch) | |
| tree | 218310f24ffb57c50c53e429e36e75a63e4d71d8 /webkit/tools | |
| parent | 7cd2fbecfcd81d19a0d028c504adfbc8f2dea0a0 (diff) | |
| download | chromium_src-d2b0ab05ce248d278b8ce4124402c631c3ec921d.zip chromium_src-d2b0ab05ce248d278b8ce4124402c631c3ec921d.tar.gz chromium_src-d2b0ab05ce248d278b8ce4124402c631c3ec921d.tar.bz2 | |
This CL fixes issue 22712 -- REGRESSION: auto-complete close cause chrome crash.
crashed was caused by a dead loop introduced by a serial webkit changes in PopupMenuClient::hidePopup/PopupDidHide.
Change AutocompletePopupMenuClient::popupDidHide() to call WebViewImpl::AutoCompletePopupDidHide() which only set the popup_showing_ flag instead of calling WebViewImpl::HideAutoCompletePopup().
BUG=http://crbug.com/22712
TEST=
1. Open the following html
<html>
<head>
</head>
<body>
<form action="form_action.asp" method="get">
First name: <input type="text" name="fname" dir="rtl"/><br />
Last name: <input type="text" name="lname" /><br />
<input type="submit" value="Submit" />
</form>
</body>
</html>
2. Type some text in the "First name" input box, for example "abcd"
3. Click "Submit"
4. Open the file again
5. Type the first letter "a" and select "abcd".
6. "abcd" should be selected instead of crashing.
Review URL: http://codereview.chromium.org/223001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@26935 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'webkit/tools')
0 files changed, 0 insertions, 0 deletions