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/search_provider_logos/logo_cache.cc b/components/search_provider_logos/logo_cache.cc
index c6272ee..6f0ae36 100644
--- a/components/search_provider_logos/logo_cache.cc
+++ b/components/search_provider_logos/logo_cache.cc
@@ -115,7 +115,7 @@ scoped_ptr<EncodedLogo> LogoCache::GetCachedLogo() {
 // static
 scoped_ptr<LogoMetadata> LogoCache::LogoMetadataFromString(
     const std::string& str, int* logo_num_bytes) {
-  scoped_ptr<base::Value> value(base::JSONReader::Read(str));
+  scoped_ptr<base::Value> value = base::JSONReader::Read(str);
   base::DictionaryValue* dict;
   if (!value || !value->GetAsDictionary(&dict))
     return scoped_ptr<LogoMetadata>();