Move away from deprecated base::Value methods in //ui

ui/app_list/search/history_data_store.cc

 // Copyright 2013 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 "ui/app_list/search/history_data_store.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <utility>
@@ skipping 127 matching lines @@
   else
     on_flushed.Run();
 }
 
 void HistoryDataStore::Load(
     const HistoryDataStore::OnLoadedCallback& on_loaded) {
   if (data_store_.get()) {
     data_store_->Load(base::Bind(
         &HistoryDataStore::OnDictionaryLoadedCallback, this, on_loaded));
   } else {
-    OnDictionaryLoadedCallback(on_loaded,
-                               base::WrapUnique(cached_dict_->DeepCopy()));
+    OnDictionaryLoadedCallback(on_loaded, cached_dict_->CreateDeepCopy());
   }
 }
 
 void HistoryDataStore::SetPrimary(const std::string& query,
                                   const std::string& result) {
   base::DictionaryValue* entry_dict = GetEntryDict(query);
-  entry_dict->SetWithoutPathExpansion(kKeyPrimary,
-                                      new base::StringValue(result));
+  entry_dict->SetStringWithoutPathExpansion(kKeyPrimary, result);
   if (data_store_.get())
     data_store_->ScheduleWrite();
 }
 
 void HistoryDataStore::SetSecondary(
     const std::string& query,
     const HistoryData::SecondaryDeque& results) {
   std::unique_ptr<base::ListValue> results_list(new base::ListValue);
   for (size_t i = 0; i < results.size(); ++i)
     results_list->AppendString(results[i]);
 
   base::DictionaryValue* entry_dict = GetEntryDict(query);
   entry_dict->SetWithoutPathExpansion(kKeySecondary, results_list.release());
   if (data_store_.get())
     data_store_->ScheduleWrite();
 }
 
 void HistoryDataStore::SetUpdateTime(const std::string& query,
                                      const base::Time& update_time) {
   base::DictionaryValue* entry_dict = GetEntryDict(query);
-  entry_dict->SetWithoutPathExpansion(kKeyUpdateTime,
-                                      new base::StringValue(base::Int64ToString(
-                                          update_time.ToInternalValue())));
+  entry_dict->SetStringWithoutPathExpansion(
+      kKeyUpdateTime, base::Int64ToString(update_time.ToInternalValue()));
   if (data_store_.get())
     data_store_->ScheduleWrite();
 }
 
 void HistoryDataStore::Delete(const std::string& query) {
   base::DictionaryValue* assoc_dict = GetAssociationDict();
   assoc_dict->RemoveWithoutPathExpansion(query, NULL);
   if (data_store_.get())
     data_store_->ScheduleWrite();
 }
 
 base::DictionaryValue* HistoryDataStore::GetAssociationDict() {
   base::DictionaryValue* cached_dict =
       cached_dict_ ? cached_dict_.get() : data_store_->cached_dict();
   DCHECK(cached_dict);
 
   base::DictionaryValue* assoc_dict = NULL;
   CHECK(cached_dict->GetDictionary(kKeyAssociations, &assoc_dict) &&
         assoc_dict);
 
   return assoc_dict;
 }
 
 base::DictionaryValue* HistoryDataStore::GetEntryDict(
     const std::string& query) {
   base::DictionaryValue* assoc_dict = GetAssociationDict();
 
-  base::DictionaryValue* entry_dict = NULL;
+  base::DictionaryValue* entry_dict = nullptr;
   if (!assoc_dict->GetDictionaryWithoutPathExpansion(query, &entry_dict)) {
     // Creates one if none exists. Ownership is taken in the set call after.
     entry_dict = new base::DictionaryValue;
-    assoc_dict->SetWithoutPathExpansion(query, entry_dict);
+    assoc_dict->SetWithoutPathExpansion(query, base::WrapUnique(entry_dict));
   }
 
   return entry_dict;
 }
 
 void HistoryDataStore::OnDictionaryLoadedCallback(
     OnLoadedCallback callback,
     std::unique_ptr<base::DictionaryValue> dict) {
   if (!dict) {
     callback.Run(nullptr);
   } else {
     callback.Run(Parse(std::move(dict)));
   }
 }
 
 }  // namespace app_list