Commit 91f3a041 authored by tzik@chromium.org's avatar tzik@chromium.org

[SyncFS] Clear sync task runners on shutdown of SyncFileSystemService

This CL should fix a memory corruption in a unit_tests which is recently introduced.
That is caused by sync task invocation after SyncFileSystemService::Shutdown.

BUG=394460
TEST=unit_tests --gtest_filter='SyncFileSystemServiceTest.*'

Review URL: https://codereview.chromium.org/395333004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@283756 0039d316-1c4b-4281-b951-d872f2087c98
parent 955667fa
...@@ -251,6 +251,9 @@ class RemoteSyncRunner : public SyncProcessRunner, ...@@ -251,6 +251,9 @@ class RemoteSyncRunner : public SyncProcessRunner,
void SyncFileSystemService::Shutdown() { void SyncFileSystemService::Shutdown() {
DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
local_sync_runners_.clear();
remote_sync_runners_.clear();
local_service_->Shutdown(); local_service_->Shutdown();
local_service_.reset(); local_service_.reset();
......
# Flaky, see http://crbug.com/394460
SyncFileSystemServiceTest.SimpleLocalSyncFlow
# Flaky, see http://crbug.com/394460
SyncFileSystemServiceTest.SimpleLocalSyncFlow
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