Convert FieldTrialList to Accept a std::unique_ptr

chrome/browser/autocomplete/search_provider_unittest.cc

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/omnibox/browser/search_provider.h"
 
 #include <stddef.h>
 
 #include <string>
 
@@ skipping 490 matching lines @@
     SCOPED_TRACE(" Case # " + base::SizeTToString(i));
     EXPECT_EQ(kNotApplicable, expected_matches[i].contents);
   }
 }
 
 void SearchProviderTest::ResetFieldTrialList() {
   // Destroy the existing FieldTrialList before creating a new one to avoid
   // a DCHECK.
   field_trial_list_.reset();
   field_trial_list_.reset(new base::FieldTrialList(
-      new metrics::SHA1EntropyProvider("foo")));
+      base::MakeUnique<metrics::SHA1EntropyProvider>("foo")));

[Alexei Svitkine (slow), 2016/09/21 15:52:00]

Nit: Actually all the test code that currently passes in an entropy provider
should just work if null is passed. So I suggest cleaning that up while you're
touching all these places. (should apply to all tests currently doing something
like this)

[robliao, 2016/09/21 17:24:29]

I haven't done an analysis of the tests to verify that this behavior is okay.

This CL shouldn't trigger any new behaviors since it's a signature change, so we
can do that in a followup CL if need be.

   variations::testing::ClearAllVariationParams();
 }
 
 base::FieldTrial* SearchProviderTest::CreateFieldTrial(
     const char* field_trial_rule,
     bool enabled) {
   std::map<std::string, std::string> params;
   params[std::string(field_trial_rule)] = enabled ?
       "true" : "false";
   variations::AssociateVariationParams(
@@ skipping 2992 matching lines @@
 }
 
 TEST_F(SearchProviderTest, DoesNotProvideOnFocus) {
   AutocompleteInput input(
       base::ASCIIToUTF16("f"), base::string16::npos, std::string(), GURL(),
       metrics::OmniboxEventProto::INVALID_SPEC, false, true, true, true, true,
       ChromeAutocompleteSchemeClassifier(&profile_));
   provider_->Start(input, false);
   EXPECT_TRUE(provider_->matches().empty());
 }