From d1199fa18e9cb795b5b88a537aef9e0327cbcabf Mon Sep 17 00:00:00 2001 From: "rlp@chromium.org" Date: Mon, 11 Jun 2012 22:47:40 +0000 Subject: Adding a RunAllPending to shortcuts_provider_unittest to fix heapcheck leaks in ShortcutsProviderTest in the case where threads are still holding on to the meassage loop. BUG=131668 TEST=heapcheck TBR=nico@chromium.org Review URL: https://chromiumcodereview.appspot.com/10539077 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141540 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/browser/autocomplete/shortcuts_provider_unittest.cc | 3 +++ 1 file changed, 3 insertions(+) (limited to 'chrome') diff --git a/chrome/browser/autocomplete/shortcuts_provider_unittest.cc b/chrome/browser/autocomplete/shortcuts_provider_unittest.cc index 097f70c..3e74385 100644 --- a/chrome/browser/autocomplete/shortcuts_provider_unittest.cc +++ b/chrome/browser/autocomplete/shortcuts_provider_unittest.cc @@ -182,6 +182,9 @@ void ShortcutsProviderTest::SetUp() { } void ShortcutsProviderTest::TearDown() { + // Run all pending tasks or else some threads hold on to the message loop + // and prevent it from being deleted. + message_loop_.RunAllPending(); provider_ = NULL; } -- cgit v1.1