Stop passing raw pointers to base::Value API in c/b/ui

chrome/browser/ui/zoom/chrome_zoom_level_prefs.cc

 // Copyright 2014 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 "chrome/browser/ui/zoom/chrome_zoom_level_prefs.h"
 
 #include <stddef.h>
 
+#include <memory>
+
 #include "base/bind.h"
+#include "base/memory/ptr_util.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/values.h"
 #include "chrome/common/chrome_constants.h"
 #include "chrome/common/pref_names.h"
 #include "components/prefs/json_pref_store.h"
 #include "components/prefs/pref_filter.h"
 #include "components/prefs/pref_registry_simple.h"
 #include "components/prefs/pref_service_factory.h"
 #include "components/prefs/scoped_user_pref_update.h"
 #include "components/zoom/zoom_event_manager.h"
@@ skipping 82 matching lines @@
     return;
   double level = change.zoom_level;
   DictionaryPrefUpdate update(pref_service_,
                               prefs::kPartitionPerHostZoomLevels);
   base::DictionaryValue* host_zoom_dictionaries = update.Get();
   DCHECK(host_zoom_dictionaries);
 
   bool modification_is_removal =
       content::ZoomValuesEqual(level, host_zoom_map_->GetDefaultZoomLevel());
 
-  base::DictionaryValue* host_zoom_dictionary = nullptr;
+  base::DictionaryValue* host_zoom_dictionary_weak = nullptr;
   if (!host_zoom_dictionaries->GetDictionary(partition_key_,
-                                             &host_zoom_dictionary)) {
-    host_zoom_dictionary = new base::DictionaryValue();
-    host_zoom_dictionaries->Set(partition_key_, host_zoom_dictionary);
+                                             &host_zoom_dictionary_weak)) {
+    auto host_zoom_dictionary = base::MakeUnique<base::DictionaryValue>();
+    host_zoom_dictionary_weak = host_zoom_dictionary.get();
+    host_zoom_dictionaries->Set(partition_key_,
+                                std::move(host_zoom_dictionary));
   }
 
-  if (modification_is_removal)
-    host_zoom_dictionary->RemoveWithoutPathExpansion(change.host, nullptr);
-  else
-    host_zoom_dictionary->SetDoubleWithoutPathExpansion(change.host, level);
+  if (modification_is_removal) {
+    host_zoom_dictionary_weak->RemoveWithoutPathExpansion(change.host, nullptr);
+  } else {
+    host_zoom_dictionary_weak->SetDoubleWithoutPathExpansion(change.host,
+                                                             level);
+  }
 }
 
 // TODO(wjmaclean): Remove the dictionary_path once the migration code is
 // removed. crbug.com/420643
 void ChromeZoomLevelPrefs::ExtractPerHostZoomLevels(
     const base::DictionaryValue* host_zoom_dictionary,
     bool sanitize_partition_host_zoom_levels) {
   std::vector<std::string> keys_to_remove;
   std::unique_ptr<base::DictionaryValue> host_zoom_dictionary_copy =
       host_zoom_dictionary->DeepCopyWithoutEmptyChildren();
@@ skipping 63 matching lines @@
                                             &host_zoom_dictionary)) {
     // Since we're calling this before setting up zoom_subscription_ below we
     // don't need to worry that host_zoom_dictionary is indirectly affected
     // by calls to HostZoomMap::SetZoomLevelForHost().
     ExtractPerHostZoomLevels(host_zoom_dictionary,
                              true /* sanitize_partition_host_zoom_levels */);
   }
   zoom_subscription_ = host_zoom_map_->AddZoomLevelChangedCallback(base::Bind(
       &ChromeZoomLevelPrefs::OnZoomLevelChanged, base::Unretained(this)));
 }