Do not flush main thread tasks in ~ScopedTaskEnvironment.

base/test/scoped_task_environment.cc

Index: base/test/scoped_task_environment.cc
diff --git a/base/test/scoped_task_environment.cc b/base/test/scoped_task_environment.cc
index d1c0907c4b0d7e32ab411826fe006c98efc3a7d6..70b1a9cc83bf11f17379d61396a3a468e80bf8ec 100644
--- a/base/test/scoped_task_environment.cc
+++ b/base/test/scoped_task_environment.cc
@@ -55,8 +55,6 @@ ScopedTaskEnvironment::ScopedTaskEnvironment(MainThreadType main_thread_type)
 }
 
 ScopedTaskEnvironment::~ScopedTaskEnvironment() {
-  RunLoop().RunUntilIdle();

[gab, 2017/05/10 15:46:49]

Replace with:

// Intentionally do not RunLoop().RunUntilIdle() here as this simulates the
replaced base::~MessageLoop() behaviour better.

[fdoray, 2017/05/10 16:01:46]

Done.

-
   DCHECK_EQ(TaskScheduler::GetInstance(), task_scheduler_);
   // Without FlushForTesting(), DeleteSoon() and ReleaseSoon() tasks could be
   // skipped, resulting in memory leaks.