Remove raw DictionaryValue::Set in //components

components/sync/syncable/syncable_id.h

 // Copyright 2012 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_SYNC_SYNCABLE_SYNCABLE_ID_H_
 #define COMPONENTS_SYNC_SYNCABLE_SYNCABLE_ID_H_
 
 #include <iosfwd>
 #include <limits>
 #include <memory>
@@ skipping 51 matching lines @@
   inline bool operator<(const Id& that) const { return s_ < that.s_; }
   inline bool operator>(const Id& that) const { return s_ > that.s_; }
 
   const std::string& value() const { return s_; }
 
   // Return the next highest ID in the lexicographic ordering.  This is
   // useful for computing upper bounds on std::sets that are ordered
   // by operator<.
   Id GetLexicographicSuccessor() const;
 
-  // Dumps the ID as a value and returns it.  Transfers ownership of
-  // the StringValue to the caller.
-  base::Value* ToValue() const;
+  // Dumps the ID as a value and returns it.
+  std::unique_ptr<base::Value> ToValue() const;
 
   // Three functions are used to work with our proto buffers.
   std::string GetServerId() const;
   static Id CreateFromServerId(const std::string& server_id);
   // This should only be used if you get back a reference to a local
   // id from the server. Returns a client only opaque id.
   static Id CreateFromClientString(const std::string& local_id);
 
   // This method returns an ID that will compare less than any valid ID.
   // The returned ID is not a valid ID itself.  This is useful for
@@ skipping 14 matching lines @@
 };
 
 inline size_t EstimateMemoryUsage(const Id& id) {
   return base::trace_event::EstimateMemoryUsage(id.value());
 }
 
 }  // namespace syncable
 }  // namespace syncer
 
 #endif  // COMPONENTS_SYNC_SYNCABLE_SYNCABLE_ID_H_