Stop passing raw pointers to base::Value API in c/b/ui

chrome/browser/ui/app_list/search/common/webservice_cache.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 "chrome/browser/ui/app_list/search/common/webservice_cache.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include "base/memory/ptr_util.h"
@@ skipping 110 matching lines @@
   base::StringToInt64(time_string, &time_val);
 
   // The result dictionary will be owned by the cache, hence create a copy
   // instead of returning the original reference. The new dictionary will be
   // owned by our MRU cache.
   *payload = Payload(base::Time::FromInternalValue(time_val),
                      base::WrapUnique(result->DeepCopy()));
   return true;
 }
 
-base::DictionaryValue* WebserviceCache::DictFromPayload(
+std::unique_ptr<base::DictionaryValue> WebserviceCache::DictFromPayload(
     const Payload& payload) {
-  base::DictionaryValue* dict = new base::DictionaryValue();
+  auto dict = base::MakeUnique<base::DictionaryValue>();
   dict->SetString(kKeyResultTime, base::Int64ToString(
       payload.time.ToInternalValue()));
   // The payload will still keep ownership of it's result dict, hence put a
   // a copy of the result dictionary here. This dictionary will be owned by
   // data_store_->cached_dict().
-  dict->Set(kKeyResult, payload.result->DeepCopy());
+  dict->Set(kKeyResult, base::MakeUnique<base::Value>(*payload.result));
 
   return dict;
 }
 
 void WebserviceCache::TrimCache() {
   for (Cache::size_type i = cache_.size(); i > kWebserviceCacheMaxSize; i--) {
     Cache::reverse_iterator rbegin = cache_.rbegin();
     data_store_->cached_dict()->Remove(rbegin->first, NULL);
     cache_.Erase(rbegin);
   }
@@ skipping 20 matching lines @@
 
 WebserviceCache::Payload::~Payload() = default;
 
 WebserviceCache::Payload& WebserviceCache::Payload::operator=(Payload&& other) {
   time = std::move(other.time);
   result = std::move(other.result);
   return *this;
 }
 
 }  // namespace app_list