Comment #15 on issue 7435 by bugdro...@chromium.org: Tab crash @
WebCore::PopupListBox::acceptIndex()
http://code.google.com/p/chromium/issues/detail?id=7435
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=9397
------------------------------------------------------------------------
r9397 | jcam...@chromium.org | 2009-02-09 11:03:40 -0800 (Mon, 09 Feb 2009)
| 10 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webview_impl.cc?r1=9397&r2=9396
M
http://src.chromium.org/viewvc/chrome/trunk/src/webkit/glue/webview_impl.h?r1=9397&r2=9396
Selecting an item in the autocomplete popup would cause a crasher.
This was caused by a change in the PopupMenuChromium where the popup would
first hide itself before setting the selection.
We use to delete our client on hide, causing a crasher when setting the
selection.
This new CL makes the autocomplete popup and client live as long as the
WebView is alive and the WebView reuse them (we just hide the popup when
necessary).
TEST=In a form, type some text in an input field. Make sure the
autocomplete popups up and behaves as expected.
BUG=7435
Review URL: http://codereview.chromium.org/21153
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---
