diff options
| author | mgist@chromium.org <mgist@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-05 19:27:15 +0000 |
|---|---|---|
| committer | mgist@chromium.org <mgist@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-05 19:27:15 +0000 |
| commit | a0e6a40637865014f5a3a00b9bee02c07cbccfa1 (patch) | |
| tree | 799e68c6e1b786f511d7aa19c31456e762076c3b /sync | |
| parent | 422e4efefbba4283f6e3b6a382a2335ec342b7b3 (diff) | |
| download | chromium_src-a0e6a40637865014f5a3a00b9bee02c07cbccfa1.zip chromium_src-a0e6a40637865014f5a3a00b9bee02c07cbccfa1.tar.gz chromium_src-a0e6a40637865014f5a3a00b9bee02c07cbccfa1.tar.bz2 | |
[Sync] Fix memory leak in TrafficRecorderTest.ToValueTimestampTest
Manage result of TrafficRecorder.ToValue() with scoped_ptr() so
memory is released when out of scope.
BUG=174209
Review URL: https://chromiumcodereview.appspot.com/12221006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180753 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'sync')
| -rw-r--r-- | sync/engine/traffic_recorder_unittest.cc | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/sync/engine/traffic_recorder_unittest.cc b/sync/engine/traffic_recorder_unittest.cc index 9c10ba5..19d682a 100644 --- a/sync/engine/traffic_recorder_unittest.cc +++ b/sync/engine/traffic_recorder_unittest.cc @@ -4,6 +4,7 @@ #include "sync/engine/traffic_recorder.h" +#include "base/memory/scoped_ptr.h" #include "base/time.h" #include "base/values.h" #include "sync/protocol/sync.pb.h" @@ -108,7 +109,9 @@ TEST(TrafficRecorderTest, ToValueTimestampTest) { recorder.set_time(sample_time); recorder.RecordClientToServerResponse(response); - ListValue* value = recorder.ToValue(); + scoped_ptr<ListValue> value; + value.reset(recorder.ToValue()); + DictionaryValue* record_value; std::string time_str; |