diff options
| author | estade <estade@chromium.org> | 2015-05-15 18:02:34 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-05-16 01:02:46 +0000 |
| commit | 8d04646294767ea1f3a7da596547b465d50cb4d1 (patch) | |
| tree | 6bd6194b6fe2ac226114f9125570bd605e9b6d42 /chrome/browser/chromeos/file_manager/url_util_unittest.cc | |
| parent | f39e5aff5aae8451a7ef190a2fc822bf80c4e30d (diff) | |
| download | chromium_src-8d04646294767ea1f3a7da596547b465d50cb4d1.zip chromium_src-8d04646294767ea1f3a7da596547b465d50cb4d1.tar.gz chromium_src-8d04646294767ea1f3a7da596547b465d50cb4d1.tar.bz2 | |
Convert JsonWriter::Write to taking a const ref for the in-param
Clearer API; flushes out a lot of unnecessary heap allocations.
depends on https://codereview.chromium.org/1129083003/
BUG=none
Review URL: https://codereview.chromium.org/1131113004
Cr-Commit-Position: refs/heads/master@{#330255}
Diffstat (limited to 'chrome/browser/chromeos/file_manager/url_util_unittest.cc')
| -rw-r--r-- | chrome/browser/chromeos/file_manager/url_util_unittest.cc | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/chrome/browser/chromeos/file_manager/url_util_unittest.cc b/chrome/browser/chromeos/file_manager/url_util_unittest.cc index 1360d00..569617d 100644 --- a/chrome/browser/chromeos/file_manager/url_util_unittest.cc +++ b/chrome/browser/chromeos/file_manager/url_util_unittest.cc @@ -23,9 +23,8 @@ std::string PrettyPrintEscapedJson(const std::string& query) { query, net::UnescapeRule::SPACES | net::UnescapeRule::URL_SPECIAL_CHARS); scoped_ptr<base::Value> value(base::JSONReader::Read(json)); std::string pretty_json; - base::JSONWriter::WriteWithOptions(value.get(), - base::JSONWriter::OPTIONS_PRETTY_PRINT, - &pretty_json); + base::JSONWriter::WriteWithOptions( + *value, base::JSONWriter::OPTIONS_PRETTY_PRINT, &pretty_json); return pretty_json; } |