diff options
author | fischman@chromium.org <fischman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-30 00:20:06 +0000 |
---|---|---|
committer | fischman@chromium.org <fischman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-30 00:20:06 +0000 |
commit | a51e9cb55e55dc40321a239ff4adfe3a52223a29 (patch) | |
tree | 13f64f8c71e151830461ad7ad97034301b5db38a /tools/heapcheck | |
parent | 7dc99087e5f341d05750fc812a1915caf96811d5 (diff) | |
download | chromium_src-a51e9cb55e55dc40321a239ff4adfe3a52223a29.zip chromium_src-a51e9cb55e55dc40321a239ff4adfe3a52223a29.tar.gz chromium_src-a51e9cb55e55dc40321a239ff4adfe3a52223a29.tar.bz2 |
Avoid leaking memory in PostTaskAndReply by Run()'ing the message_loop.
Turns out that PostTaskAndReply to a runner that never Run()s triggers
a leak. See http://crbug.com/120289 for details.
BUG=119714
TEST=Cmd line that uses suppressions before this CL and doesn't after: ./tools/heapcheck/chrome_tests.sh --build_dir ninja/Debug --test media --gtest_filter=AudioInput*
Review URL: http://codereview.chromium.org/9858027
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129735 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'tools/heapcheck')
-rw-r--r-- | tools/heapcheck/suppressions.txt | 14 |
1 files changed, 0 insertions, 14 deletions
diff --git a/tools/heapcheck/suppressions.txt b/tools/heapcheck/suppressions.txt index ed6fd8c..361b9ee 100644 --- a/tools/heapcheck/suppressions.txt +++ b/tools/heapcheck/suppressions.txt @@ -1796,17 +1796,3 @@ ... fun:SelectFileDialogExtensionTest_SelfDeleting_Test::TestBody } -{ - bug_119714_a - Heapcheck:Leak - ... - fun:*AudioInputControllerTest_CreateAndClose_Test* -} -{ - # This should be removed soon! It's a pretty broad suppression. - bug_119714_b - Heapcheck:Leak - fun:MessageLoop::MessageLoop - fun:base::Thread::ThreadMain - fun:base::::ThreadFunc -} |