Chromium Code Reviews| Index: chrome/browser/autocomplete/autocomplete_popup_model.cc |
| diff --git a/chrome/browser/autocomplete/autocomplete_popup_model.cc b/chrome/browser/autocomplete/autocomplete_popup_model.cc |
| index 588683dc4b96148ec578265f0f47516c8cd3aae8..d62272bd3c524dc123845c5798f09d8b6ced40e9 100644 |
| --- a/chrome/browser/autocomplete/autocomplete_popup_model.cc |
| +++ b/chrome/browser/autocomplete/autocomplete_popup_model.cc |
| @@ -317,8 +317,15 @@ void AutocompletePopupModel::Observe(NotificationType type, |
| if ((hovered_line_ != kNoMatch) && (result->size() <= hovered_line_)) |
| SetHoveredLine(kNoMatch); |
| + const bool was_open = view_->IsOpen(); |
| view_->UpdatePopupAppearance(); |
| - edit_model_->PopupBoundsChangedTo(view_->GetTargetBounds()); |
| + |
|
Peter Kasting
2011/01/24 22:10:19
Nit: Unnecessary blank line
|
| + if (view_->IsOpen()) { |
| + edit_model_->PopupBoundsChangedTo(view_->GetTargetBounds()); |
| + } else if (was_open) { |
| + edit_model_->PopupBoundsChangedTo(gfx::Rect()); |
|
Peter Kasting
2011/01/24 22:10:19
Nit: Let's roll this call into the body of OnPopup
|
| + edit_model_->OnPopupClosed(); |
| + } |
| } |
| const SkBitmap* AutocompletePopupModel::GetSpecialIconForMatch( |