Remove uses of deprecated RegisterDictionaryPref

The override of RegisterDictionaryPref 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 dictionary (as there is an override doing this). If the dictionary
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: I0b6028fd9ac2a27ae9ba72d3af3c0d6a7d478301
Reviewed-on: https://chromium-review.googlesource.com/c/1454486Reviewed-by: Julian Pastarmov <pastarmovj@chromium.org>
Commit-Queue: Sylvain Defresne <sdefresne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#629171}

chrome/browser/chromeos/policy/device_status_collector.cc

@@ -1094,15 +1094,13 @@ DeviceStatusCollector::~DeviceStatusCollector() {
 
 // static
 void DeviceStatusCollector::RegisterPrefs(PrefRegistrySimple* registry) {
-  registry->RegisterDictionaryPref(prefs::kDeviceActivityTimes,
-                                   std::make_unique<base::DictionaryValue>());
+  registry->RegisterDictionaryPref(prefs::kDeviceActivityTimes);
 }
 
 // static
 void DeviceStatusCollector::RegisterProfilePrefs(PrefRegistrySimple* registry) {
   registry->RegisterBooleanPref(prefs::kReportArcStatusEnabled, false);
-  registry->RegisterDictionaryPref(prefs::kUserActivityTimes,
-                                   std::make_unique<base::DictionaryValue>());
+  registry->RegisterDictionaryPref(prefs::kUserActivityTimes);
   registry->RegisterTimePref(prefs::kLastChildScreenTimeReset, Time());
   registry->RegisterTimePref(prefs::kLastChildScreenTimeSaved, Time());
   registry->RegisterIntegerPref(prefs::kChildScreenTimeMilliseconds, 0);