Use scoped_ptrs in JSONReader::Read functions.

There are many callers, so all could not be updated at once. The old version is renamed to JSONReader::DeprecatedRead, and a new version of JSONReader::Read that returns scoped_ptr takes its place.

Much of this patch was generated with sed. Some callsites of the form

  scoped_ptr<Value> value(Read());

have been updated to

  scoped_ptr<Value> value = Read();

but most Read() calls are simply converted to DeprecatedRead. Actually updating them is a TODO.

BUG=none

Review URL: https://codereview.chromium.org/1136643005

Cr-Commit-Position: refs/heads/master@{#331120}

1 files changed, 1 insertions, 1 deletions

diff --git a/components/invalidation/p2p_invalidator.cc b/components/invalidation/p2p_invalidator.cc
index e6c03b5..aea1e3a 100644
--- a/components/invalidation/p2p_invalidator.cc
+++ b/components/invalidation/p2p_invalidator.cc
@@ -112,7 +112,7 @@ std::string P2PNotificationData::ToString() const {
 }
 
 bool P2PNotificationData::ResetFromString(const std::string& str) {
-  scoped_ptr<base::Value> data_value(base::JSONReader::Read(str));
+  scoped_ptr<base::Value> data_value = base::JSONReader::Read(str);
   const base::DictionaryValue* data_dict = NULL;
   if (!data_value.get() || !data_value->GetAsDictionary(&data_dict)) {
     LOG(WARNING) << "Could not parse " << str << " as a dictionary";