Remove raw DictionaryValue::Set in //components

components/cloud_devices/common/printer_description.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 "components/cloud_devices/common/printer_description.h"
 
 #include <stddef.h>
 
 #include <algorithm>
 #include <memory>
 #include <utility>
 
 #include "base/json/json_reader.h"
 #include "base/json/json_writer.h"
 #include "base/macros.h"
+#include "base/memory/ptr_util.h"
 #include "base/strings/string_util.h"
 #include "base/values.h"
 #include "components/cloud_devices/common/cloud_device_description_consts.h"
 #include "components/cloud_devices/common/description_items_inl.h"
 
 namespace cloud_devices {
 
 namespace printer {
 
 namespace {
@@ skipping 700 matching lines @@
       Interval new_interval(1, kMaxPageNumber);
       interval->GetInteger(kPageRangeStart, &new_interval.start);
       interval->GetInteger(kPageRangeEnd, &new_interval.end);
       option->push_back(new_interval);
     }
     return true;
   }
 
   static void Save(const PageRange& option, base::DictionaryValue* dict) {
     if (!option.empty()) {
-      base::ListValue* list = new base::ListValue;
-      dict->Set(kPageRangeInterval, list);
+      auto list = base::MakeUnique<base::ListValue>();
       for (size_t i = 0; i < option.size(); ++i) {
-        std::unique_ptr<base::DictionaryValue> interval(
-            new base::DictionaryValue);
+        auto interval = base::MakeUnique<base::DictionaryValue>();
         interval->SetInteger(kPageRangeStart, option[i].start);
         if (option[i].end < kMaxPageNumber)
           interval->SetInteger(kPageRangeEnd, option[i].end);
         list->Append(std::move(interval));
       }
+      dict->Set(kPageRangeInterval, std::move(list));
     }
   }
 };
 
 class MediaTraits : public ItemsTraits<kOptionMediaSize> {
  public:
   static bool IsValid(const Media& option) { return option.IsValid(); }
 
   static bool Load(const base::DictionaryValue& dict, Media* option) {
     std::string type_str;
@@ skipping 79 matching lines @@
 template class TicketItem<Margins, MarginsTraits>;
 template class TicketItem<Dpi, DpiTraits>;
 template class TicketItem<FitToPageType, FitToPageTraits>;
 template class TicketItem<Media, MediaTraits>;
 template class TicketItem<int32_t, CopiesTraits>;
 template class TicketItem<PageRange, PageRangeTraits>;
 template class TicketItem<bool, CollateTraits>;
 template class TicketItem<bool, ReverseTraits>;
 
 }  // namespace cloud_devices