commit d8ca4d19d8c82c8bd6367a26e7f91b05580e8b08
author ahendrickson@chromium.org <ahendrickson@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Thu Apr 07 22:05:13 2011
committer ahendrickson@chromium.org <ahendrickson@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Thu Apr 07 22:05:13 2011
tree 841a1c87760e15a4307c287d1a3bfff86ff4f2c0
parent 0b895b76cb6de14ef75ea30538d10b1d01687f13

Cleaned up memory leak in Download Manager unit tests.

BUG=None
TEST=Download manager unit tests no longer leak in Valgrind.

Review URL: http://codereview.chromium.org/6815015

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@80845 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/download/download_manager_unittest.cc

diff --git a/chrome/browser/download/download_manager_unittest.cc b/chrome/browser/download/download_manager_unittest.cc
index 1f1cd7c..dde3db3 100644
--- a/chrome/browser/download/download_manager_unittest.cc
+++ b/chrome/browser/download/download_manager_unittest.cc
@@ -430,7 +430,7 @@
   ASSERT_TRUE(download != NULL);
 
   EXPECT_EQ(DownloadItem::IN_PROGRESS, download->state());
-  ItemObserver* observer = new ItemObserver(download);
+  scoped_ptr<ItemObserver> observer(new ItemObserver(download));
 
   download_file->AppendDataToFile(kTestData, kTestDataLen);
 
@@ -497,7 +497,7 @@
   ASSERT_TRUE(download != NULL);
 
   EXPECT_EQ(DownloadItem::IN_PROGRESS, download->state());
-  ItemObserver* observer = new ItemObserver(download);
+  scoped_ptr<ItemObserver> observer(new ItemObserver(download));
 
   info->path = new_path;
   AttachDownloadItem(info);