Skip to content

Commit

Permalink
Fix mem leak in DownloadUpdatesCommandTest
Browse files Browse the repository at this point in the history
The leak was introduced in r199136.

BUG=231693,138613

Review URL: https://chromiumcodereview.appspot.com/15001016

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@199355 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
rlarocque@chromium.org committed May 10, 2013
1 parent c862271 commit 9c86108
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions sync/engine/download_updates_command_unittest.cc
Original file line number Diff line number Diff line change
Expand Up @@ -45,11 +45,12 @@ TEST_F(DownloadUpdatesCommandTest, ExecuteNoStates) {

mock_server()->ExpectGetUpdatesRequestTypes(
GetRoutingInfoTypes(routing_info()));
command_.ExecuteImpl(
scoped_ptr<sessions::SyncSession> session(
sessions::SyncSession::BuildForNudge(context(),
delegate(),
nudge_tracker.GetSourceInfo(),
&nudge_tracker));
command_.ExecuteImpl(session.get());
}

TEST_F(DownloadUpdatesCommandTest, ExecuteWithStates) {
Expand All @@ -70,11 +71,12 @@ TEST_F(DownloadUpdatesCommandTest, ExecuteWithStates) {
GetRoutingInfoTypes(routing_info()));
mock_server()->ExpectGetUpdatesRequestStates(
nudge_tracker.GetSourceInfo().types);
command_.ExecuteImpl(
scoped_ptr<sessions::SyncSession> session(
sessions::SyncSession::BuildForNudge(context(),
delegate(),
nudge_tracker.GetSourceInfo(),
&nudge_tracker));
command_.ExecuteImpl(session.get());
}

TEST_F(DownloadUpdatesCommandTest, VerifyAppendDebugInfo) {
Expand Down

0 comments on commit 9c86108

Please sign in to comment.