tree 473f3c12069aab30079d1d17f80d675f3ca97d41
parent e8adf59b7aa651287f2825df819576086e783f97
author Alexander Curtiss <curtissa@google.com> 1495669683 -0700
committer chrome-bot <chrome-bot@chromium.org> 1497911591 -0700

dbus_send: Fixed a bug with _build_arg_string()

The function would incorrectly error when a numeric dbus.* type object
was passed in (e.g. dbus.Int16) due to a 'continue' statement within a
nested loop. This was bypassed by refactoring the loop body.

BUG=chromium:726432
TEST=dbus_send_unittest.py

Change-Id: Ic8028746fe2e4cc5ae459c91bea28208d675bacf
Reviewed-on: https://chromium-review.googlesource.com/514409
Commit-Ready: Alexander Curtiss <curtissa@google.com>
Tested-by: Kirtika Ruchandani <kirtika@chromium.org>
Tested-by: Alexander Curtiss <curtissa@google.com>
Reviewed-by: Eric Caruso <ejcaruso@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>