Remove raw DictionaryValue::Set in //components

components/bookmarks/browser/bookmark_codec.h

 // 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.
 
 #ifndef COMPONENTS_BOOKMARKS_BROWSER_BOOKMARK_CODEC_H_
 #define COMPONENTS_BOOKMARKS_BROWSER_BOOKMARK_CODEC_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 24 matching lines @@
   // are not unique, we will reassign IDs to make them unique. There are no
   // guarantees on how the IDs are reassigned or about doing minimal
   // reassignments to achieve uniqueness.
   BookmarkCodec();
   ~BookmarkCodec();
 
   // Encodes the model to a JSON value. It's up to the caller to delete the
   // returned object. This is invoked to encode the contents of the bookmark bar
   // model and is currently a convenience to invoking Encode that takes the
   // bookmark bar node and other folder node.
-  base::Value* Encode(BookmarkModel* model);
+  std::unique_ptr<base::Value> Encode(BookmarkModel* model);
 
-  // Encodes the bookmark bar and other folders returning the JSON value. It's
-  // up to the caller to delete the returned object.
-  base::Value* Encode(const BookmarkNode* bookmark_bar_node,
-                      const BookmarkNode* other_folder_node,
-                      const BookmarkNode* mobile_folder_node,
-                      const BookmarkNode::MetaInfoMap* model_meta_info_map,
-                      int64_t sync_transaction_version);
+  // Encodes the bookmark bar and other folders returning the JSON value.
+  std::unique_ptr<base::Value> Encode(
+      const BookmarkNode* bookmark_bar_node,
+      const BookmarkNode* other_folder_node,
+      const BookmarkNode* mobile_folder_node,
+      const BookmarkNode::MetaInfoMap* model_meta_info_map,
+      int64_t sync_transaction_version);
 
   // Decodes the previously encoded value to the specified nodes as well as
   // setting |max_node_id| to the greatest node id. Returns true on success,
   // false otherwise. If there is an error (such as unexpected version) all
   // children are removed from the bookmark bar and other folder nodes. On exit
   // |max_node_id| is set to the max id of the nodes.
   bool Decode(BookmarkNode* bb_node,
               BookmarkNode* other_folder_node,
               BookmarkNode* mobile_folder_node,
               int64_t* max_node_id,
@@ skipping 41 matching lines @@
   static const char kSyncTransactionVersion[];
 
   // Possible values for kTypeKey.
   static const char kTypeURL[];
   static const char kTypeFolder[];
 
  private:
   // Encodes node and all its children into a Value object and returns it.
   std::unique_ptr<base::Value> EncodeNode(const BookmarkNode* node);
 
-  // Encodes the given meta info into a Value object and returns it. The caller
-  // takes ownership of the returned object.
-  base::Value* EncodeMetaInfo(const BookmarkNode::MetaInfoMap& meta_info_map);
+  // Encodes the given meta info into a Value object and returns it.
+  std::unique_ptr<base::Value> EncodeMetaInfo(
+      const BookmarkNode::MetaInfoMap& meta_info_map);
 
   // Helper to perform decoding.
   bool DecodeHelper(BookmarkNode* bb_node,
                     BookmarkNode* other_folder_node,
                     BookmarkNode* mobile_folder_node,
                     const base::Value& value);
 
   // Decodes the children of the specified node. Returns true on success.
   bool DecodeChildren(const base::ListValue& child_value_list,
                       BookmarkNode* parent);
@@ skipping 76 matching lines @@
 
   // Sync transaction version set on bookmark model root.
   int64_t model_sync_transaction_version_;
 
   DISALLOW_COPY_AND_ASSIGN(BookmarkCodec);
 };
 
 }  // namespace bookmarks
 
 #endif  // COMPONENTS_BOOKMARKS_BROWSER_BOOKMARK_CODEC_H_