Fix initialization for a few more unit_tests under PlzNavigate

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 ca4aad9ab2e16c745acefc2b2b9eb225719fa65d..381a1934909cb2250ff61a37489125acdb699c3b 100644
--- a/chrome/browser/ui/webui/settings/metrics_reporting_handler_unittest.cc
+++ b/chrome/browser/ui/webui/settings/metrics_reporting_handler_unittest.cc
@@ -35,7 +35,6 @@ class MetricsReportingHandlerTest : public testing::Test {
  public:
   MetricsReportingHandlerTest() {
     // Local state must be set up before |handler_|.
-    TestingBrowserProcess::CreateInstance();

[clamy, 2016/08/30 21:23:21]

Why do we need to remove this, and isn't it a problem if we do it?

[scottmg, 2016/08/30 22:03:15]

It's initialized twice at the moment, here, but also very early on in
https://cs.chromium.org/chromium/src/chrome/test/base/chrome_unit_test_suite....
.

I guess no mainline bots notice because the whole file is in #if
GOOGLE_CHROME_BUILD, only happened to notice because I run with that locally on
my Windows box. So... not too important either way, I guess.

     local_state_.reset(new ScopedTestingLocalState(
         TestingBrowserProcess::GetGlobal()));