Commit 28dd846f authored by Greg Thompson's avatar Greg Thompson Committed by Commit Bot

Plug resource leaks in GCMDriverBaseTest.

Unregistering ends up posting tasks to perform cleanup in the Encryption
db. Let these tasks run during TearDown so that all files have been
closed by the time the test fixture's ScopedTempDir is torn down.

BUG=546640

Change-Id: Iddbe8857e96c3042ea9d15223b299acb521a1b6e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1943230
Auto-Submit: Greg Thompson <grt@chromium.org>
Commit-Queue: Peter Beverloo <peter@chromium.org>
Reviewed-by: default avatarPeter Beverloo <peter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#720179}
parent 71da3cbb
......@@ -213,6 +213,7 @@ void GCMDriverTest::TearDown() {
io_thread_.Stop();
task_environment_.RunUntilIdle();
ASSERT_TRUE(temp_dir_.Delete());
}
void GCMDriverTest::PumpIOLoop() {
......
......@@ -151,6 +151,8 @@ void GCMDriverBaseTest::TearDown() {
PumpIOLoop();
io_thread_.Stop();
task_environment_.RunUntilIdle();
ASSERT_TRUE(temp_dir_.Delete());
}
void GCMDriverBaseTest::PumpIOLoop() {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment