MD Settings: fix metrics reporting unittests on official testers -- take 2

chrome/browser/ui/webui/settings/metrics_reporting_handler_unittest.cc

Index: chrome/browser/ui/webui/settings/metrics_reporting_handler_unittest.cc
diff --git a/chrome/browser/ui/webui/settings/metrics_reporting_handler_unittest.cc b/chrome/browser/ui/webui/settings/metrics_reporting_handler_unittest.cc
index 038cd6a33b55ed70f9d5707809502d4a63209099..adbb52c74c8b6846410c72ba1eebaa7ae6b0efa9 100644
--- a/chrome/browser/ui/webui/settings/metrics_reporting_handler_unittest.cc
+++ b/chrome/browser/ui/webui/settings/metrics_reporting_handler_unittest.cc
@@ -61,7 +61,11 @@ class MetricsReportingHandlerTest : public testing::Test {
   }
 
   void TearDown() override {
+    base::RunLoop().RunUntilIdle();
     handler_.reset();
+    base::RunLoop().RunUntilIdle();
+    local_state_.reset();
+    base::RunLoop().RunUntilIdle();

[dschuyler, 2016/08/19 21:14:04]

imo, this begs for a comment about what
the goal is. Maybe a TODO style comment?

[Dan Beam, 2016/08/24 19:17:56]

Done.

   }
 
   PrefService* local_state() { return local_state_->Get(); }