diff options
| author | estade <estade@chromium.org> | 2015-05-22 09:30:13 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-05-22 16:30:52 +0000 |
| commit | b09312bb5ad1bffaad8a09deea395f1aef2053df (patch) | |
| tree | ea13602ed858503180cd725d1afe6a4ce73a2bd9 /components/domain_reliability | |
| parent | 2d381b0799c9fbbd33fbfa5ccf6610794d454f71 (diff) | |
| download | chromium_src-b09312bb5ad1bffaad8a09deea395f1aef2053df.zip chromium_src-b09312bb5ad1bffaad8a09deea395f1aef2053df.tar.gz chromium_src-b09312bb5ad1bffaad8a09deea395f1aef2053df.tar.bz2 | |
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}
Diffstat (limited to 'components/domain_reliability')
| -rw-r--r-- | components/domain_reliability/config.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/components/domain_reliability/config.cc b/components/domain_reliability/config.cc index 7aaf9a5..6f47154 100644 --- a/components/domain_reliability/config.cc +++ b/components/domain_reliability/config.cc @@ -96,7 +96,7 @@ scoped_ptr<const DomainReliabilityConfig> DomainReliabilityConfig::FromJSON( tracked_objects::ScopedTracker tracking_profile( FROM_HERE_WITH_EXPLICIT_FUNCTION( "436671 DomainReliabilityConfig::FromJSON")); - scoped_ptr<base::Value> value(base::JSONReader::Read(json)); + scoped_ptr<base::Value> value = base::JSONReader::Read(json); base::JSONValueConverter<DomainReliabilityConfig> converter; scoped_ptr<DomainReliabilityConfig> config(new DomainReliabilityConfig()); |