Use unique_ptr<DictionaryValue> in ProxyConfigDictionary

components/proxy_config/proxy_config_dictionary.h

 // Copyright (c) 2011 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.
 
 #ifndef COMPONENTS_PROXY_CONFIG_PROXY_CONFIG_DICTIONARY_H_
 #define COMPONENTS_PROXY_CONFIG_PROXY_CONFIG_DICTIONARY_H_
 
 #include <memory>
 #include <string>
 
@@ skipping 13 matching lines @@
 // in the user preferences. The dictionary has the following structure:
 // {
 //   mode: string,
 //   server: string,
 //   pac_url: string,
 //   bypass_list: string
 // }
 // See proxy_config_dictionary.cc for the structure of the respective strings.
 class PROXY_CONFIG_EXPORT ProxyConfigDictionary {
  public:
-  // Creates a deep copy of |dict| and leaves ownership to caller.
-  explicit ProxyConfigDictionary(const base::DictionaryValue* dict);
+  explicit ProxyConfigDictionary(std::unique_ptr<base::DictionaryValue> dict);
   ~ProxyConfigDictionary();
 
   bool GetMode(ProxyPrefs::ProxyMode* out) const;
   bool GetPacUrl(std::string* out) const;
   bool GetPacMandatory(bool* out) const;
   bool GetProxyServer(std::string* out) const;
   bool GetBypassList(std::string* out) const;
   bool HasBypassList() const;
 
   const base::DictionaryValue& GetDictionary() const;
 
-  static base::DictionaryValue* CreateDirect();
-  static base::DictionaryValue* CreateAutoDetect();
-  static base::DictionaryValue* CreatePacScript(const std::string& pac_url,
-                                                bool pac_mandatory);
-  static base::DictionaryValue* CreateFixedServers(
+  static std::unique_ptr<base::DictionaryValue> CreateDirect();
+  static std::unique_ptr<base::DictionaryValue> CreateAutoDetect();
+  static std::unique_ptr<base::DictionaryValue> CreatePacScript(
+      const std::string& pac_url,
+      bool pac_mandatory);
+  static std::unique_ptr<base::DictionaryValue> CreateFixedServers(
       const std::string& proxy_server,
       const std::string& bypass_list);
-  static base::DictionaryValue* CreateSystem();
+  static std::unique_ptr<base::DictionaryValue> CreateSystem();
 
   // Encodes the proxy server as "<url-scheme>=<proxy-scheme>://<proxy>".
   // Used to generate the |proxy_server| arg for CreateFixedServers().
   static void EncodeAndAppendProxyServer(const std::string& url_scheme,
                                          const net::ProxyServer& server,
                                          std::string* spec);
 
  private:
-  static base::DictionaryValue* CreateDictionary(
+  static std::unique_ptr<base::DictionaryValue> CreateDictionary(
       ProxyPrefs::ProxyMode mode,
       const std::string& pac_url,
       bool pac_mandatory,
       const std::string& proxy_server,
       const std::string& bypass_list);
 
   std::unique_ptr<base::DictionaryValue> dict_;
 
   DISALLOW_COPY_AND_ASSIGN(ProxyConfigDictionary);
 };
 
 #endif  // COMPONENTS_PROXY_CONFIG_PROXY_CONFIG_DICTIONARY_H_