diff options
| author | sergeyu@chromium.org <sergeyu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-18 19:18:30 +0000 |
|---|---|---|
| committer | sergeyu@chromium.org <sergeyu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-06-18 19:18:30 +0000 |
| commit | 31fd701637d04ae727433cfd216a6fadc2c859e6 (patch) | |
| tree | e6a29e27bb25d62a6646486e212e6625e07524b2 | |
| parent | f5743daf2a1580a1a4e123ff998d4072110165f7 (diff) | |
| download | chromium_src-31fd701637d04ae727433cfd216a6fadc2c859e6.zip chromium_src-31fd701637d04ae727433cfd216a6fadc2c859e6.tar.gz chromium_src-31fd701637d04ae727433cfd216a6fadc2c859e6.tar.bz2 | |
Fixed memleak in JsonHostConfig.
BUG=46882
TEST=heapcheck on remoting_unittests
Review URL: http://codereview.chromium.org/2819013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@50270 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | remoting/host/json_host_config.cc | 1 | ||||
| -rw-r--r-- | tools/heapcheck/suppressions.txt | 21 |
2 files changed, 1 insertions, 21 deletions
diff --git a/remoting/host/json_host_config.cc b/remoting/host/json_host_config.cc index 324b2d2..ecb54b9 100644 --- a/remoting/host/json_host_config.cc +++ b/remoting/host/json_host_config.cc @@ -40,6 +40,7 @@ void JsonHostConfig::Update(Task* task) { AutoLock auto_lock(lock_); task->Run(); } + delete task; message_loop_proxy_->PostTask( FROM_HERE, NewRunnableMethod(this, &JsonHostConfig::DoWrite)); } diff --git a/tools/heapcheck/suppressions.txt b/tools/heapcheck/suppressions.txt index bb71d38..0d22f51 100644 --- a/tools/heapcheck/suppressions.txt +++ b/tools/heapcheck/suppressions.txt @@ -761,27 +761,6 @@ } { - bug_46882_a - Heapcheck:Leak - ... - fun:remoting::JsonHostConfig::Read -} - -{ - bug_46882_b - Heapcheck:Leak - ... - fun:remoting::JsonHostConfigTest_Write_Test::TestBody -} - -{ - bug_46882_c - Heapcheck:Leak - fun:base::MessageLoopProxy::CreateForCurrentThread - fun:remoting::JsonHostConfigTest::SetUp -} - -{ bug_46886 Heapcheck:Leak ... |