Remove raw DictionaryValue::Set in //components

components/sync/syncable/model_type.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 "components/sync/base/model_type.h"
 
 #include <stddef.h>
 
 #include "base/macros.h"
+#include "base/memory/ptr_util.h"
 #include "base/strings/string_split.h"
 #include "base/values.h"
 #include "components/reading_list/core/reading_list_enable_flags.h"
 #include "components/sync/protocol/app_notification_specifics.pb.h"
 #include "components/sync/protocol/app_setting_specifics.pb.h"
 #include "components/sync/protocol/app_specifics.pb.h"
 #include "components/sync/protocol/autofill_specifics.pb.h"
 #include "components/sync/protocol/bookmark_specifics.pb.h"
 #include "components/sync/protocol/extension_setting_specifics.pb.h"
 #include "components/sync/protocol/extension_specifics.pb.h"
@@ skipping 535 matching lines @@
 // assigned.
 //
 // Don't forget to update the "SyncModelTypes" enum in histograms.xml when you
 // make changes to this list.
 int ModelTypeToHistogramInt(ModelType model_type) {
   if (model_type >= UNSPECIFIED && model_type < MODEL_TYPE_COUNT)
     return kModelTypeInfoMap[model_type].model_type_histogram_val;
   return 0;
 }
 
-base::Value* ModelTypeToValue(ModelType model_type) {
+std::unique_ptr<base::Value> ModelTypeToValue(ModelType model_type) {
   if (model_type >= FIRST_REAL_MODEL_TYPE) {
-    return new base::Value(ModelTypeToString(model_type));
+    return base::MakeUnique<base::Value>(ModelTypeToString(model_type));
   } else if (model_type == TOP_LEVEL_FOLDER) {
-    return new base::Value("Top-level folder");
+    return base::MakeUnique<base::Value>("Top-level folder");
   } else if (model_type == UNSPECIFIED) {
-    return new base::Value("Unspecified");
+    return base::MakeUnique<base::Value>("Unspecified");
   }
   NOTREACHED();
-  return new base::Value(std::string());
+  return base::MakeUnique<base::Value>(std::string());
 }
 
 ModelType ModelTypeFromValue(const base::Value& value) {
   if (value.IsType(base::Value::Type::STRING)) {
     std::string result;
     CHECK(value.GetAsString(&result));
     return ModelTypeFromString(result);
   } else if (value.IsType(base::Value::Type::INTEGER)) {
     int result;
     CHECK(value.GetAsInteger(&result));
@@ skipping 140 matching lines @@
 bool TypeSupportsHierarchy(ModelType model_type) {
   // TODO(stanisc): crbug/438313: Should this also include TOP_LEVEL_FOLDER?
   return model_type == BOOKMARKS;
 }
 
 bool TypeSupportsOrdering(ModelType model_type) {
   return model_type == BOOKMARKS;
 }
 
 }  // namespace syncer