Remove raw base::DictionaryValue::Set

ios/chrome/browser/notification_promo.cc

 // Copyright (c) 2016 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 "ios/chrome/browser/notification_promo.h"
 
 #include <stddef.h>
 
+#include <utility>
+
+#include "base/memory/ptr_util.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
 #include "base/time/time.h"
 #include "base/values.h"
 #include "components/pref_registry/pref_registry_syncable.h"
 #include "components/prefs/pref_registry_simple.h"
 #include "components/prefs/pref_service.h"
 #include "components/variations/variations_associated_data.h"
 
 namespace ios {
@@ skipping 50 matching lines @@
     // Choose the dictionary to which parameter is added based on whether the
     // parameter belongs in the payload or not.
     base::DictionaryValue& json_or_payload =
         IsPayloadParam(iter->first) ? payload : json;
     if (converted) {
       json_or_payload.SetInteger(iter->first, converted_number);
     } else {
       json_or_payload.SetString(iter->first, iter->second);
     }
   }
-  json.Set("payload", payload.DeepCopy());
+  json.Set("payload", base::MakeUnique<base::Value>(payload));
 
   InitFromJson(json);
 }
 
 void NotificationPromo::InitFromJson(const base::DictionaryValue& promo) {
   std::string time_str;
   base::Time time;
   if (promo.GetString("start", &time_str) &&
       base::Time::FromString(time_str.c_str(), &time)) {
     start_ = time.ToDoubleT();
@@ skipping 42 matching lines @@
 }
 
 void NotificationPromo::WritePrefs() {
   WritePrefs(promo_id_, first_view_time_, views_, closed_);
 }
 
 void NotificationPromo::WritePrefs(int promo_id,
                                    double first_view_time,
                                    int views,
                                    bool closed) {
-  base::DictionaryValue* ntp_promo = new base::DictionaryValue;
+  auto ntp_promo = base::MakeUnique<base::DictionaryValue>();
   ntp_promo->SetDouble(kPrefPromoFirstViewTime, first_view_time);
   ntp_promo->SetInteger(kPrefPromoViews, views);
   ntp_promo->SetBoolean(kPrefPromoClosed, closed);
 
   base::DictionaryValue promo_dict;
   promo_dict.MergeDictionary(local_state_->GetDictionary(kPrefPromoObject));
-  promo_dict.Set(base::IntToString(promo_id), ntp_promo);
+  promo_dict.Set(base::IntToString(promo_id), std::move(ntp_promo));
   local_state_->Set(kPrefPromoObject, promo_dict);
   DVLOG(1) << "WritePrefs " << promo_dict;
 }
 
 void NotificationPromo::InitFromPrefs() {
   // Check if data is stored in the old prefs structure, and migrate it before
   // reading from prefs.
   MigrateOldPrefs();
 
   // If |promo_id_| is not set, do nothing.
@@ skipping 100 matching lines @@
 
 double NotificationPromo::StartTime() const {
   return start_;
 }
 
 double NotificationPromo::EndTime() const {
   return end_;
 }
 
 }  // namespace ios