Fix memory leak in unit test

BUG=70388 TEST=PrintingContextTest.PrintAll Review URL: http://codereview.chromium.org/7795031 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@100102 0039d316-1c4b-4281-b951-d872f2087c98
dednal · Sep 8, 2011 · e7f96d1 · e7f96d1
1 parent 61f900b
commit e7f96d1
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/printing/printing_context_win.cc b/printing/printing_context_win.cc
@@ -256,6 +256,7 @@ void PrintingContextWin::AskUserForSettings(gfx::NativeView view,
 
   // TODO(maruel):  Support PD_PRINTTOFILE.
   callback->Run(ParseDialogResultEx(dialog_options));
+  delete callback;
 #endif
 }
 

diff --git a/tools/valgrind/drmemory/suppressions.txt b/tools/valgrind/drmemory/suppressions.txt
@@ -746,12 +746,6 @@ LEAK
 *!net::SSL*SocketNSS::DoHandshake
 *!net::SSL*SocketNSS::DoHandshakeLoop
 
-# http://crbug.com/70388
-LEAK
-*!operator new
-*!NewCallback<PrintingContextTest,enum printing::PrintingContext::Result>
-*!PrintingContextTest_PrintAll_Test::TestBody
-
 # http://crbug.com/73744
 LEAK
 *!operator new
-Original file line number
+Diff line change
@@ Expand Up @@
       // TODO(maruel):  Support PD_PRINTTOFILE.
       callback->Run(ParseDialogResultEx(dialog_options));
+      delete callback;
     #endif
     }
@@ Expand Down @@