Remove raw base::DictionaryValue::SetWithoutPathExpansion in //chrome

chrome/browser/safe_browsing/incident_reporting/state_store.cc

 // Copyright 2015 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/safe_browsing/incident_reporting/state_store.h"
 
 #include <utility>
 
+#include "base/memory/ptr_util.h"
 #include "base/metrics/histogram_macros.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/values.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/safe_browsing/incident_reporting/incident.h"
 #include "chrome/browser/safe_browsing/incident_reporting/platform_state_store.h"
 #include "chrome/common/pref_names.h"
 #include "components/prefs/pref_service.h"
 
 namespace safe_browsing {
@@ skipping 16 matching lines @@
 }
 
 void StateStore::Transaction::MarkAsReported(IncidentType type,
                                              const std::string& key,
                                              IncidentDigest digest) {
   std::string type_string(base::IntToString(static_cast<int32_t>(type)));
   base::DictionaryValue* incidents_sent = GetPrefDict();
   base::DictionaryValue* type_dict = nullptr;
   if (!incidents_sent->GetDictionaryWithoutPathExpansion(type_string,
                                                          &type_dict)) {
-    type_dict = new base::DictionaryValue();
-    incidents_sent->SetWithoutPathExpansion(type_string, type_dict);
+    incidents_sent->SetWithoutPathExpansion(
+        type_string, base::MakeUnique<base::DictionaryValue>());
+    incidents_sent->GetDictionaryWithoutPathExpansion(type_string, &type_dict);
   }
   type_dict->SetStringWithoutPathExpansion(key, base::UintToString(digest));
 }
 
 void StateStore::Transaction::Clear(IncidentType type, const std::string& key) {
   // Nothing to do if the pref dict does not exist.
   if (!store_->incidents_sent_)
     return;
 
   // Use the read-only view on the preference to figure out if there is a value
@@ skipping 112 matching lines @@
   static const IncidentType kLegacyTypes[] = {
       // TODO(grt): remove in M44 (crbug.com/451173).
       IncidentType::OMNIBOX_INTERACTION,
   };
 
   for (IncidentType type : kLegacyTypes)
     transaction->ClearForType(type);
 }
 
 }  // namespace safe_browsing