WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
35172
[Chromium] Use the m_suggestionsPopupClient pointer to refer to the popup client, as the Autocomplete popup might not be the current popup
https://bugs.webkit.org/show_bug.cgi?id=35172
Summary
[Chromium] Use the m_suggestionsPopupClient pointer to refer to the popup cli...
James Hawkins
Reported
2010-02-19 13:30:53 PST
This fixes a crash when the AutoFill suggestions popup is shown and the user deletes enough characters so that the popup hides itself. Patch will follow.
Attachments
[Chromium] Use the m_suggestionsPopupClient pointer to refer to the popup client, as the Autocomplete popup might not be the current popup
(2.04 KB, patch)
2010-02-19 13:32 PST
,
James Hawkins
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
James Hawkins
Comment 1
2010-02-19 13:32:11 PST
Created
attachment 49102
[details]
[Chromium] Use the m_suggestionsPopupClient pointer to refer to the popup client, as the Autocomplete popup might not be the current popup
Eric Seidel (no email)
Comment 2
2010-02-19 14:17:12 PST
Comment on
attachment 49102
[details]
[Chromium] Use the m_suggestionsPopupClient pointer to refer to the popup client, as the Autocomplete popup might not be the current popup What other popups (non-autocomplete) are we using? Some sort of grammar/spelling?
James Hawkins
Comment 3
2010-02-19 14:18:46 PST
The AutoFill popup is the other popup. It behaves a bit differently than the Autocomplete popup.
WebKit Commit Bot
Comment 4
2010-02-19 14:51:00 PST
Comment on
attachment 49102
[details]
[Chromium] Use the m_suggestionsPopupClient pointer to refer to the popup client, as the Autocomplete popup might not be the current popup Clearing flags on attachment: 49102 Committed
r55032
: <
http://trac.webkit.org/changeset/55032
>
WebKit Commit Bot
Comment 5
2010-02-19 14:51: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.
Top of Page
Format For Printing
XML
Clone This Bug