Remove uses of deprecated RegisterListPref

The override of RegisterListPref taking a std::unique_ptr<base::Value>
is deprecated in favor of the method taking the base::Value by value.

Convert client code to either not pass default value if the value is just
an empty list (as there is an override doing this). If the list is not
empty, convert the code to create and pass it by value.

This CL was uploaded by git cl split.

Bug: 646113
Change-Id: Iba4c00eacafe077995ee73f6bd970bb8f927251f
Reviewed-on: https://chromium-review.googlesource.com/c/1454369
Auto-Submit: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Mihai Sardarescu <msarda@chromium.org>
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#629539}

components/signin/core/browser/signin_manager_base.cc

@@ -51,8 +51,7 @@ void SigninManagerBase::RegisterProfilePrefs(PrefRegistrySimple* registry) {
   registry->RegisterStringPref(prefs::kGoogleServicesUserAccountId,
                                std::string());
   registry->RegisterBooleanPref(prefs::kAutologinEnabled, true);
-  registry->RegisterListPref(prefs::kReverseAutologinRejectedEmailList,
-                             std::make_unique<base::ListValue>());
+  registry->RegisterListPref(prefs::kReverseAutologinRejectedEmailList);
   registry->RegisterBooleanPref(prefs::kSigninAllowed, true);
   registry->RegisterInt64Pref(prefs::kSignedInTime,
                               base::Time().ToInternalValue());