Use std::unique_ptr for base::DictionaryValue and base::ListValue's internal store.

chrome/browser/devtools/devtools_embedder_message_dispatcher.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/devtools/devtools_embedder_message_dispatcher.h"
 
 #include "base/bind.h"
 #include "base/values.h"
 
 namespace {
 
 using DispatchCallback = DevToolsEmbedderMessageDispatcher::DispatchCallback;
 
-bool GetValue(const base::Value* value, std::string* result) {
-  return value->GetAsString(result);
+bool GetValue(const base::Value& value, std::string* result) {
+  return value.GetAsString(result);
 }
 
-bool GetValue(const base::Value* value, int* result) {
-  return value->GetAsInteger(result);
+bool GetValue(const base::Value& value, int* result) {
+  return value.GetAsInteger(result);
 }
 
-bool GetValue(const base::Value* value, bool* result) {
-  return value->GetAsBoolean(result);
+bool GetValue(const base::Value& value, bool* result) {
+  return value.GetAsBoolean(result);
 }
 
-bool GetValue(const base::Value* value, gfx::Rect* rect) {
+bool GetValue(const base::Value& value, gfx::Rect* rect) {
   const base::DictionaryValue* dict;
-  if (!value->GetAsDictionary(&dict))
+  if (!value.GetAsDictionary(&dict))
     return false;
   int x = 0;
   int y = 0;
   int width = 0;
   int height = 0;
   if (!dict->GetInteger("x", &x) ||
       !dict->GetInteger("y", &y) ||
       !dict->GetInteger("width", &width) ||
       !dict->GetInteger("height", &height))
     return false;
@@ skipping 21 matching lines @@
   template <typename H, typename... As>
   void Apply(const H& handler, As... args) {
     handler.Run(args...);
   }
 };
 
 template <typename T, typename... Ts>
 struct ParamTuple<T, Ts...> {
   bool Parse(const base::ListValue& list,
              const base::ListValue::const_iterator& it) {
-    return it != list.end() && GetValue(*it, &head) && tail.Parse(list, it + 1);
+    return it != list.end() && GetValue(**it, &head) &&
+           tail.Parse(list, it + 1);
   }
 
   template <typename H, typename... As>
   void Apply(const H& handler, As... args) {
     tail.template Apply<H, As..., T>(handler, args..., head);
   }
 
   typename StorageTraits<T>::StorageType head;
   ParamTuple<Ts...> tail;
 };
@@ skipping 124 matching lines @@
   d->RegisterHandler("setPreference",
                      &Delegate::SetPreference, delegate);
   d->RegisterHandler("removePreference",
                      &Delegate::RemovePreference, delegate);
   d->RegisterHandler("clearPreferences",
                      &Delegate::ClearPreferences, delegate);
   d->RegisterHandler("readyForTest",
                      &Delegate::ReadyForTest, delegate);
   return d;
 }