diff options
| author | estade <estade@chromium.org> | 2015-05-07 14:16:45 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-05-07 21:17:10 +0000 |
| commit | c6bb7fcb643e4ca240f793e1455155ea2699ce5b (patch) | |
| tree | f3b8bc8bb8b617f3cf4be22be81bb8b7b9f93aa0 /sync/internal_api/change_record.cc | |
| parent | 61cae85448cfeb793270e804b5ad1023993279c5 (diff) | |
| download | chromium_src-c6bb7fcb643e4ca240f793e1455155ea2699ce5b.zip chromium_src-c6bb7fcb643e4ca240f793e1455155ea2699ce5b.tar.gz chromium_src-c6bb7fcb643e4ca240f793e1455155ea2699ce5b.tar.bz2 | |
Convert some parts of sync/ to use of scoped_ptr instead of bare ptrs
for base::Value handling.
BUG=none
Review URL: https://codereview.chromium.org/1126913002
Cr-Commit-Position: refs/heads/master@{#328830}
Diffstat (limited to 'sync/internal_api/change_record.cc')
| -rw-r--r-- | sync/internal_api/change_record.cc | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/sync/internal_api/change_record.cc b/sync/internal_api/change_record.cc index 4894b3c..d6c7d36 100644 --- a/sync/internal_api/change_record.cc +++ b/sync/internal_api/change_record.cc @@ -17,8 +17,8 @@ ChangeRecord::ChangeRecord() ChangeRecord::~ChangeRecord() {} -base::DictionaryValue* ChangeRecord::ToValue() const { - base::DictionaryValue* value = new base::DictionaryValue(); +scoped_ptr<base::DictionaryValue> ChangeRecord::ToValue() const { + scoped_ptr<base::DictionaryValue> value(new base::DictionaryValue()); std::string action_str; switch (action) { case ACTION_ADD: @@ -55,7 +55,8 @@ ExtraPasswordChangeRecordData::ExtraPasswordChangeRecordData( ExtraPasswordChangeRecordData::~ExtraPasswordChangeRecordData() {} -base::DictionaryValue* ExtraPasswordChangeRecordData::ToValue() const { +scoped_ptr<base::DictionaryValue> ExtraPasswordChangeRecordData::ToValue() + const { return PasswordSpecificsDataToValue(unencrypted_); } |