diff options
author | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-10 20:07:37 +0000 |
---|---|---|
committer | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-08-10 20:07:37 +0000 |
commit | 0988a53d7b523148ef762dc9809812bfd7853c02 (patch) | |
tree | 146251ee4c701fa7a7868c59796907c72d432c0a /chrome/browser/gtk/task_manager_gtk.cc | |
parent | 1aff9b7acb4f40dc23daa43b98405f621d8b9a62 (diff) | |
download | chromium_src-0988a53d7b523148ef762dc9809812bfd7853c02.zip chromium_src-0988a53d7b523148ef762dc9809812bfd7853c02.tar.gz chromium_src-0988a53d7b523148ef762dc9809812bfd7853c02.tar.bz2 |
gtk: Don't crash if the user double clicks an empty area of the task manager tree view.
BUG=18960
TEST=Double click the empty space of the task manager tree view. The browser should not crash.
Review URL: http://codereview.chromium.org/165253
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22942 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/gtk/task_manager_gtk.cc')
-rw-r--r-- | chrome/browser/gtk/task_manager_gtk.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/chrome/browser/gtk/task_manager_gtk.cc b/chrome/browser/gtk/task_manager_gtk.cc index 99958f3..e6ad400 100644 --- a/chrome/browser/gtk/task_manager_gtk.cc +++ b/chrome/browser/gtk/task_manager_gtk.cc @@ -552,8 +552,10 @@ void TaskManagerGtk::ActivateFocusedTab() { // If the user has just double clicked, only one item is selected. GtkTreeModel* model; GList* selected = gtk_tree_selection_get_selected_rows(selection, &model); - int row = GetRowNumForPath(reinterpret_cast<GtkTreePath*>(selected->data)); - task_manager_->ActivateProcess(row); + if (selected) { + int row = GetRowNumForPath(reinterpret_cast<GtkTreePath*>(selected->data)); + task_manager_->ActivateProcess(row); + } } void TaskManagerGtk::OnLinkActivated() { |