diff options
| author | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-17 22:17:20 +0000 |
|---|---|---|
| committer | pkasting@chromium.org <pkasting@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-17 22:17:20 +0000 |
| commit | 3b98cc6462be77713a1027582181a597424b6916 (patch) | |
| tree | ad61159d069523ba65def73b55332620d794b136 /chrome | |
| parent | a5b58f5ca9c9a179cfaba0ff237f12b5ca9d7de8 (diff) | |
| download | chromium_src-3b98cc6462be77713a1027582181a597424b6916.zip chromium_src-3b98cc6462be77713a1027582181a597424b6916.tar.gz chromium_src-3b98cc6462be77713a1027582181a597424b6916.tar.bz2 | |
Fix DCHECK when trying to remove multiple engines from the Manage Search Engines dialog. Original patch by Thiago Farina (see http://codereview.chromium.org/401013 ), r=me.
BUG=5227
TEST=Select multiple engines and hit "Remove" in a debug build; should not checkfail.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32223 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
| -rw-r--r-- | chrome/browser/views/keyword_editor_view.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/views/keyword_editor_view.cc b/chrome/browser/views/keyword_editor_view.cc index 9e4446b..ed1b938 100644 --- a/chrome/browser/views/keyword_editor_view.cc +++ b/chrome/browser/views/keyword_editor_view.cc @@ -246,7 +246,7 @@ void KeywordEditorView::ButtonPressed( browser::EditSearchEngine(GetWindow()->GetNativeWindow(), NULL, this, profile_); } else if (sender == remove_button_) { - DCHECK_EQ(1, table_view_->SelectedRowCount()); + DCHECK_GT(table_view_->SelectedRowCount(), 0); int last_view_row = -1; for (views::TableView::iterator i = table_view_->SelectionBegin(); i != table_view_->SelectionEnd(); ++i) { |