[code health] Replace base::DictionaryValue::Iterator
Replace base::DictionaryValue::Iterator with base::Value::DictItems(). Bug: 1187056 Change-Id: I34ff6cb3b4c31f30dee972653a92c1f5418b81a7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3388869 Reviewed-by: Ken Rockot <rockot@google.com> Commit-Queue: Weiliang Chen <weiliangc@chromium.org> Cr-Commit-Position: refs/heads/main@{#959442}
This commit is contained in:

committed by
Chromium LUCI CQ

parent
65f00e3b81
commit
7395b2c617
@ -123,15 +123,12 @@ void WriteValue(base::Pickle* m, const base::Value* value, int recursion) {
|
||||
break;
|
||||
}
|
||||
case base::Value::Type::DICTIONARY: {
|
||||
const base::DictionaryValue* dict =
|
||||
static_cast<const base::DictionaryValue*>(value);
|
||||
DCHECK(value->is_dict());
|
||||
WriteParam(m, base::checked_cast<int>(value->DictSize()));
|
||||
|
||||
WriteParam(m, base::checked_cast<int>(dict->DictSize()));
|
||||
|
||||
for (base::DictionaryValue::Iterator it(*dict); !it.IsAtEnd();
|
||||
it.Advance()) {
|
||||
WriteParam(m, it.key());
|
||||
WriteValue(m, &it.value(), recursion + 1);
|
||||
for (auto it : value->DictItems()) {
|
||||
WriteParam(m, it.first);
|
||||
WriteValue(m, &it.second, recursion + 1);
|
||||
}
|
||||
break;
|
||||
}
|
||||
|
Reference in New Issue
Block a user