Remove raw DictionaryValue::Set in //chrome

chrome/browser/ssl/chrome_ssl_host_state_delegate.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/ssl/chrome_ssl_host_state_delegate.h"
 
 #include <stdint.h>
 
 #include <set>
 #include <string>
 #include <utility>
 
 #include "base/base64.h"
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/command_line.h"
 #include "base/guid.h"
 #include "base/logging.h"
+#include "base/memory/ptr_util.h"
 #include "base/metrics/field_trial.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/time/clock.h"
 #include "base/time/default_clock.h"
 #include "base/time/time.h"
 #include "base/values.h"
 #include "chrome/browser/content_settings/host_content_settings_map_factory.h"
 #include "chrome/browser/profiles/profile.h"
 #include "components/content_settings/core/browser/host_content_settings_map.h"
 #include "components/content_settings/core/common/content_settings_types.h"
@@ skipping 227 matching lines @@
 
   dict->SetString(kSSLCertDecisionGUIDKey, current_expiration_guid_);
 
   // Extract the map of certificate fingerprints to errors from the setting.
   base::DictionaryValue* cert_error_dict = NULL;  // Will be owned by dict
   if (expired ||
       !dict->GetDictionary(kSSLCertDecisionCertErrorMapKey, &cert_error_dict)) {
     if (create_entries == DO_NOT_CREATE_DICTIONARY_ENTRIES)
       return NULL;
 
-    cert_error_dict = new base::DictionaryValue();
-    // dict takes ownership of cert_error_dict
-    dict->Set(kSSLCertDecisionCertErrorMapKey, cert_error_dict);
+    cert_error_dict =
+        dict->SetDictionary(kSSLCertDecisionCertErrorMapKey,
+                            base::MakeUnique<base::DictionaryValue>());
   }
 
   return cert_error_dict;
 }
 
 // If |should_remember_ssl_decisions_| is
 // FORGET_SSL_EXCEPTION_DECISIONS_AT_SESSION_END, that means that all invalid
 // certificate proceed decisions should be forgotten when the session ends. To
 // simulate that, Chrome keeps track of a guid to represent the current browser
 // session and stores it in decision entries. See the comment for
@@ skipping 207 matching lines @@
     case CERT_ERRORS_CONTENT:
       return !!ran_content_with_cert_errors_hosts_.count(
           BrokenHostEntry(host, child_id));
   }
   NOTREACHED();
   return false;
 }
 void ChromeSSLHostStateDelegate::SetClock(std::unique_ptr<base::Clock> clock) {
   clock_ = std::move(clock);
 }