chromium/src
0
Fork 0
Code Activity

[Code Health] Replace Value::GetAsList with GetList

Browse Source 
This is part of the great base::Value refactoring, and fixes same
remaining uses in dbus/values_util.cc of GetAsList.

Bug: 1187009
Change-Id: Ib9d706d8c3f50d42b3166539b606aec6fb8e14d3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3351971
Reviewed-by: Ryo Hashimoto <hashimoto@chromium.org>
Commit-Queue: Viktor Semeniuk <vsemeniuk@google.com>
Cr-Commit-Position: refs/heads/main@{#953525}
This commit is contained in:
Viktor Semeniuk
2021-12-22 14:30:24 +00:00
committed by Chromium LUCI CQ
parent 31a9bb3985
commit 78efb67fbd
1 changed files with 1 additions and 3 deletions

4
dbus/values_util.cc

@ -267,11 +267,9 @@ void AppendValueData(MessageWriter* writer, const base::Value& value) {
break;
}
case base::Value::Type::LIST: {
const base::ListValue* list = nullptr;
value.GetAsList(&list);
dbus::MessageWriter array_writer(nullptr);
writer->OpenArray("v", &array_writer);
for (const auto& value_in_list : list->GetList()) {
for (const auto& value_in_list : value.GetList()) {
AppendValueDataAsVariant(&array_writer, value_in_list);
}
writer->CloseContainer(&array_writer);