diff options
author | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-24 23:49:18 +0000 |
---|---|---|
committer | jcampan@chromium.org <jcampan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-24 23:49:18 +0000 |
commit | 55578b060daacab5d7767640114e25cc3886116f (patch) | |
tree | 5b02037854170c080920121bfffb8ecf7b8b0210 /chrome/browser/views/task_manager_view.cc | |
parent | c4cb846563fa083485ef7eb177f4c91d3df7d69f (diff) | |
download | chromium_src-55578b060daacab5d7767640114e25cc3886116f.zip chromium_src-55578b060daacab5d7767640114e25cc3886116f.tar.gz chromium_src-55578b060daacab5d7767640114e25cc3886116f.tar.bz2 |
Now that the tab_table_ of the TaskManager is parent owned, it should not be accessed after the views have been removed.
This was causing a crasher in the browser tests (as a deleted object was accessed).
BUG=None
TEST=Run the browser tests.
Review URL: http://codereview.chromium.org/147110
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19198 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/views/task_manager_view.cc')
-rw-r--r-- | chrome/browser/views/task_manager_view.cc | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/chrome/browser/views/task_manager_view.cc b/chrome/browser/views/task_manager_view.cc index 32fe975..ec80055 100644 --- a/chrome/browser/views/task_manager_view.cc +++ b/chrome/browser/views/task_manager_view.cc @@ -270,10 +270,6 @@ TaskManagerView::TaskManagerView() TaskManagerView::~TaskManagerView() { // Delete child views now, while our table model still exists. RemoveAllChildViews(true); - - // Prevent the table from accessing the model as part of its destruction, as - // the model might already be destroyed. - tab_table_->SetModel(NULL); } void TaskManagerView::Init() { |