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..6c3f9c4329a91128fb37d9db9c06b2a70151e2dc 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,12 @@ class MetricsReportingHandlerTest : public testing::Test {
   }
 
   void TearDown() override {
+    // For crbug.com/637068 which only run on official bots with no try jobs.

[dschuyler, 2016/08/24 22:14:59]

I suggest ignoring this optional nit:
but just to be thorough s/run/runs/

+    base::RunLoop().RunUntilIdle();
     handler_.reset();
+    base::RunLoop().RunUntilIdle();
+    local_state_.reset();
+    base::RunLoop().RunUntilIdle();
   }
 
   PrefService* local_state() { return local_state_->Get(); }