Change the signature of JSONReader::Read() and related methods to be more

chrome/browser/dom_ui/dom_ui.cc

 // Copyright (c) 2006-2008 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/dom_ui/dom_ui.h"
 
 #include "base/json_reader.h"
 #include "base/json_writer.h"
 #include "chrome/common/l10n_util.h"
 
@@ skipping 13 matching lines @@
 
 void DOMUI::ProcessDOMUIMessage(const std::string& message,
                                 const std::string& content) {
   // Look up the callback for this message.
   MessageCallbackMap::const_iterator callback = 
       message_callbacks_.find(message);
   if (callback == message_callbacks_.end())
     return;
 
   // Convert the content JSON into a Value.
-  Value* value = NULL;
+  scoped_ptr<Value> value;
   if (!content.empty()) {
-    if (!JSONReader::Read(content, &value, false)) {
+    value.reset(JSONReader::Read(content, false));
+    if (!value.get()) {
       // The page sent us something that we didn't understand.
       // This probably indicates a programming error.
       NOTREACHED();
       return;
     }
   }
 
   // Forward this message and content on.
-  callback->second->Run(value);
-  delete value;
+  callback->second->Run(value.get());
 }
 
 void DOMUI::CallJavascriptFunction(const std::wstring& function_name,
                                    const Value& arg) {
   std::string json;
   JSONWriter::Write(&arg, false, &json);
   std::wstring javascript = function_name + L"(" + UTF8ToWide(json) + L");";
 
   ExecuteJavascript(javascript);
 }
@@ skipping 58 matching lines @@
     if (using_url_as_the_title) {
       l10n_util::WrapStringWithLTRFormatting(&title_to_set);
     } else {
       bool success =
           l10n_util::AdjustStringForLocaleDirection(title, &title_to_set);
       DCHECK(success ? (title != title_to_set) : (title == title_to_set));
     }
   }
   dictionary->SetString(L"title", title_to_set);
 }