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

chrome/browser/safe_browsing/incident_reporting/incident_reporting_service_unittest.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/safe_browsing/incident_reporting/incident_reporting_service.h"
 
 #include <stdint.h>
 
 #include <map>
 #include <string>
 #include <utility>
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/lazy_instance.h"
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "base/test/mock_entropy_provider.h"
 #include "base/test/scoped_feature_list.h"
 #include "base/test/scoped_mock_time_message_loop_task_runner.h"
 #include "base/threading/thread_local.h"
 #include "base/threading/thread_task_runner_handle.h"
+#include "base/values.h"
 #include "build/build_config.h"
 #include "chrome/browser/prefs/browser_prefs.h"
 #include "chrome/browser/safe_browsing/incident_reporting/incident.h"
 #include "chrome/browser/safe_browsing/incident_reporting/incident_receiver.h"
 #include "chrome/browser/safe_browsing/incident_reporting/incident_report_uploader.h"
 #include "chrome/browser/safe_browsing/incident_reporting/last_download_finder.h"
 #include "chrome/browser/safe_browsing/incident_reporting/tracked_preference_incident.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/test/base/testing_browser_process.h"
 #include "chrome/test/base/testing_profile.h"
@@ skipping 1271 matching lines @@
 TEST_F(IncidentReportingServiceTest, CleanLegacyPruneState) {
   CreateIncidentReportingService();
   const std::string omnibox_type(base::IntToString(
       static_cast<int32_t>(safe_browsing::IncidentType::OMNIBOX_INTERACTION)));
   const std::string preference_type(base::IntToString(
       static_cast<int32_t>(safe_browsing::IncidentType::TRACKED_PREFERENCE)));
 
   // Set up a prune state dict with data to be cleared (and not).
   std::unique_ptr<base::DictionaryValue> incidents_sent(
       new base::DictionaryValue());
-  base::DictionaryValue* type_dict = new base::DictionaryValue();
+  auto type_dict = base::MakeUnique<base::DictionaryValue>();
   type_dict->SetStringWithoutPathExpansion("foo", "47");
-  incidents_sent->SetWithoutPathExpansion(omnibox_type, type_dict);
-  type_dict = new base::DictionaryValue();
+  incidents_sent->SetWithoutPathExpansion(omnibox_type, std::move(type_dict));
+  type_dict = base::MakeUnique<base::DictionaryValue>();
   type_dict->SetStringWithoutPathExpansion("bar", "43");
-  incidents_sent->SetWithoutPathExpansion(preference_type, type_dict);
+  incidents_sent->SetWithoutPathExpansion(preference_type,
+                                          std::move(type_dict));
 
   // Add a profile.
   Profile* profile =
       CreateProfile("profile1", SAFE_BROWSING_AND_EXTENDED_REPORTING,
                     ON_PROFILE_ADDITION_NO_ACTION, std::move(incidents_sent));
 
   // Let all tasks run.
   mock_time_task_runner_->FastForwardUntilNoTasksRemain();
 
   const base::DictionaryValue* new_state =
@@ skipping 70 matching lines @@
 
   // Ensure that no report processing remains.
   ASSERT_FALSE(instance_->IsProcessingReport());
 }
 
 // Parallel uploads
 // Shutdown during processing
 // environment colection taking longer than incident delay timer
 // environment colection taking longer than incident delay timer, and then
 // another incident arriving