Remove raw DictionaryValue::Set in //components

components/sync/engine_impl/js_mutation_event_observer.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/engine_impl/js_mutation_event_observer.h"
 
 #include <stddef.h>
+#include <utility>
 
 #include "base/location.h"
 #include "base/logging.h"
+#include "base/memory/ptr_util.h"
 #include "base/strings/string_number_conversions.h"
 #include "base/values.h"
 #include "components/sync/js/js_event_details.h"
 #include "components/sync/js/js_event_handler.h"
 
 namespace syncer {
 
 JsMutationEventObserver::JsMutationEventObserver() : weak_ptr_factory_(this) {}
 
 JsMutationEventObserver::~JsMutationEventObserver() {
@@ skipping 25 matching lines @@
     ModelType model_type,
     int64_t write_transaction_id,
     const ImmutableChangeRecordList& changes) {
   if (!event_handler_.IsInitialized()) {
     return;
   }
   base::DictionaryValue details;
   details.SetString("modelType", ModelTypeToString(model_type));
   details.SetString("writeTransactionId",
                     base::Int64ToString(write_transaction_id));
-  base::Value* changes_value = nullptr;
+  std::unique_ptr<base::Value> changes_value;
   const size_t changes_size = changes.Get().size();
   if (changes_size <= kChangeLimit) {
-    base::ListValue* changes_list = new base::ListValue();
+    auto changes_list = base::MakeUnique<base::ListValue>();
     for (ChangeRecordList::const_iterator it = changes.Get().begin();
          it != changes.Get().end(); ++it) {
       changes_list->Append(it->ToValue());
     }
-    changes_value = changes_list;
+    changes_value = std::move(changes_list);
   } else {
-    changes_value =
-        new base::Value(base::SizeTToString(changes_size) + " changes");
+    changes_value = base::MakeUnique<base::Value>(
+        base::SizeTToString(changes_size) + " changes");
   }
-  details.Set("changes", changes_value);
+  details.Set("changes", std::move(changes_value));
   HandleJsEvent(FROM_HERE, "onChangesApplied", JsEventDetails(&details));
 }
 
 void JsMutationEventObserver::OnChangesComplete(ModelType model_type) {
   if (!event_handler_.IsInitialized()) {
     return;
   }
   base::DictionaryValue details;
   details.SetString("modelType", ModelTypeToString(model_type));
   HandleJsEvent(FROM_HERE, "onChangesComplete", JsEventDetails(&details));
@@ skipping 18 matching lines @@
     const std::string& name,
     const JsEventDetails& details) {
   if (!event_handler_.IsInitialized()) {
     NOTREACHED();
     return;
   }
   event_handler_.Call(from_here, &JsEventHandler::HandleJsEvent, name, details);
 }
 
 }  // namespace syncer