diff options
| author | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-08 01:00:03 +0000 |
|---|---|---|
| committer | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-08 01:00:03 +0000 |
| commit | b2936a488c205a22953cac658e768af13ce8b953 (patch) | |
| tree | 61409e04c937d9426fa4820aa89d7738a6fc2dc3 | |
| parent | f5e4b9bf073a0d3cb7fa67ed14448ec9a936ff6b (diff) | |
| download | chromium_src-b2936a488c205a22953cac658e768af13ce8b953.zip chromium_src-b2936a488c205a22953cac658e768af13ce8b953.tar.gz chromium_src-b2936a488c205a22953cac658e768af13ce8b953.tar.bz2 | |
Revert sanity check; this doesn't actually work since UpdatePopupAppearance() never removes child views (the popup quickly grows to its child limit and then stays there forever; we only show some of the children at a time though).
BUG=20511
TEST=none
TBR=huanr
Review URL: http://codereview.chromium.org/262018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28361 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | chrome/browser/views/autocomplete/autocomplete_popup_contents_view.cc | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/chrome/browser/views/autocomplete/autocomplete_popup_contents_view.cc b/chrome/browser/views/autocomplete/autocomplete_popup_contents_view.cc index 474f45e..ea1908e 100644 --- a/chrome/browser/views/autocomplete/autocomplete_popup_contents_view.cc +++ b/chrome/browser/views/autocomplete/autocomplete_popup_contents_view.cc @@ -619,9 +619,7 @@ gfx::Rect AutocompletePopupContentsView::GetPopupBounds() const { // AutocompletePopupContentsView, AutocompletePopupView overrides: bool AutocompletePopupContentsView::IsOpen() const { - const bool is_open = popup_->IsOpen(); - CHECK(!!GetChildViewCount() == is_open); - return is_open; + return popup_->IsOpen(); } void AutocompletePopupContentsView::InvalidateLine(size_t line) { |