Remove raw DictionaryValue::Set in //components

chrome/browser/bookmarks/bookmark_html_writer.cc

 // Copyright (c) 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.
 
 #include "chrome/browser/bookmarks/bookmark_html_writer.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <memory>
@@ skipping 78 matching lines @@
 const char kFolderChildren[] = "<DL><p>";
 // End of the children for a folder.
 const char kFolderChildrenEnd[] = "</DL><p>";
 
 // Number of characters to indent by.
 const size_t kIndentSize = 4;
 
 // Class responsible for the actual writing. Takes ownership of favicons_map.
 class Writer : public base::RefCountedThreadSafe<Writer> {
  public:
-  Writer(base::Value* bookmarks,
+  Writer(std::unique_ptr<base::Value> bookmarks,
          const base::FilePath& path,
          BookmarkFaviconFetcher::URLFaviconMap* favicons_map,
          BookmarksExportObserver* observer)
-      : bookmarks_(bookmarks),
+      : bookmarks_(std::move(bookmarks)),
         path_(path),
         favicons_map_(favicons_map),
-        observer_(observer) {
-  }
+        observer_(observer) {}
 
   // Writing bookmarks and favicons data to file.
   void DoWrite() {
     if (!OpenFile())
       return;
 
     base::Value* roots = NULL;
     if (!Write(kHeader) ||
         bookmarks_->GetType() != base::Value::Type::DICTIONARY ||
         !static_cast<base::DictionaryValue*>(bookmarks_.get())->Get(
@@ skipping 397 matching lines @@
   // BookmarkModel isn't thread safe (nor would we want to lock it down
   // for the duration of the write), as such we make a copy of the
   // BookmarkModel using BookmarkCodec then write from that.
   if (!g_fetcher) {
     g_fetcher = new BookmarkFaviconFetcher(profile, path, observer);
     g_fetcher->ExportBookmarks();
   }
 }
 
 }  // namespace bookmark_html_writer