diff options
| author | mdm@chromium.org <mdm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-07-28 22:13:14 +0000 |
|---|---|---|
| committer | mdm@chromium.org <mdm@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-07-28 22:13:14 +0000 |
| commit | 07fd407c339c5d0469da916f22c718dbbf2f6746 (patch) | |
| tree | 65802b3a3f247c16c4b05f3c913ce480eda44b8d /chrome/browser/gtk | |
| parent | 7aaabfdfa7052d7c128cdbf9cebe62358b1e0739 (diff) | |
| download | chromium_src-07fd407c339c5d0469da916f22c718dbbf2f6746.zip chromium_src-07fd407c339c5d0469da916f22c718dbbf2f6746.tar.gz chromium_src-07fd407c339c5d0469da916f22c718dbbf2f6746.tar.bz2 | |
Linux: it's not actually a bug to have the selection become empty in the URL picker.
BUG=none
TEST=in a debug build, go to wrench->options->basics, select "open the following pages", click "add", select something from the list, and then deselect it by holding ctrl and clicking it again; chromium should not crash
Review URL: http://codereview.chromium.org/159540
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@21902 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/gtk')
| -rw-r--r-- | chrome/browser/gtk/options/url_picker_dialog_gtk.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/chrome/browser/gtk/options/url_picker_dialog_gtk.cc b/chrome/browser/gtk/options/url_picker_dialog_gtk.cc index 7042853..d6bf0eb 100644 --- a/chrome/browser/gtk/options/url_picker_dialog_gtk.cc +++ b/chrome/browser/gtk/options/url_picker_dialog_gtk.cc @@ -308,7 +308,7 @@ void UrlPickerDialogGtk::OnHistorySelectionChanged( GtkTreeSelection *selection, UrlPickerDialogGtk* window) { GtkTreeIter iter; if (!gtk_tree_selection_get_selected(selection, NULL, &iter)) { - NOTREACHED(); + // The user has unselected the history element, nothing to do. return; } GtkTreePath* path = gtk_tree_model_get_path( |