0

Remove shadow variable names from dbus/

This was causing some -Wshadow compiler errors in Chromecast builds.

Change-Id: I2ebdd629e3472302db5d4d05e340c89223d690eb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2197490
Reviewed-by: Ryo Hashimoto <hashimoto@chromium.org>
Commit-Queue: Matt Swartwout <mwswartwout@google.com>
Auto-Submit: Matt Swartwout <mwswartwout@google.com>
Cr-Commit-Position: refs/heads/master@{#769032}
This commit is contained in:
Matt Swartwout
2020-05-14 23:14:01 +00:00
committed by Commit Bot
parent 16ca64a05b
commit af6cc997f9
2 changed files with 2 additions and 3 deletions

@ -561,7 +561,6 @@ DBusHandlerResult ObjectProxy::HandleMessage(DBusConnection* connection,
FROM_HERE, base::BindOnce(&ObjectProxy::RunMethod, this, start_time,
iter->second, released_signal));
} else {
const base::TimeTicks start_time = base::TimeTicks::Now();
// If the D-Bus thread is not used, just call the callback on the
// current thread. Transfer the ownership of |signal| to RunMethod().
Signal* released_signal = signal.release();

@ -281,8 +281,8 @@ void AppendValueData(MessageWriter* writer, const base::Value& value) {
value.GetAsList(&list);
dbus::MessageWriter array_writer(nullptr);
writer->OpenArray("v", &array_writer);
for (const auto& value : *list) {
AppendValueDataAsVariant(&array_writer, value);
for (const auto& value_in_list : *list) {
AppendValueDataAsVariant(&array_writer, value_in_list);
}
writer->CloseContainer(&array_writer);
break;