Fix BLE register Dbus interactions.

Fix BLE register Dbus interactions.

Currently the data going to BlueZ for the advertising calls is not in the
correct format. This CL fixes various issues causing problems.

R=armansito@chromium.org, jamuraa@chromium.org
BUG=496988
Committed: https://crrev.com/ab23b18bc2937ff716e42805105fe417c829b60c
Cr-Commit-Position: refs/heads/master@{#338933}

[armansito, 2015-07-09 21:03:35]

https://codereview.chromium.org/1223173008/diff/1/chromeos/dbus/bluetooth_le_...
File chromeos/dbus/bluetooth_le_advertisement_service_provider.cc (right):

https://codereview.chromium.org/1223173008/diff/1/chromeos/dbus/bluetooth_le_...
chromeos/dbus/bluetooth_le_advertisement_service_provider.cc:261: void
AppendArrayVariantOfStrings(dbus::MessageWriter* dict_writer,
Do you mean AppendVariantArrayOfStrings?

https://codereview.chromium.org/1223173008/diff/1/chromeos/dbus/bluetooth_le_...
chromeos/dbus/bluetooth_le_advertisement_service_provider.cc:266:
strings_array_variant.OpenArray("s", &strings_array);
You should be able to just call strings_array_variant.AppendArrayOfStrings

https://codereview.chromium.org/1223173008/diff/1/device/bluetooth/bluetooth_...
File device/bluetooth/bluetooth_advertisement_chromeos.cc (right):

https://codereview.chromium.org/1223173008/diff/1/device/bluetooth/bluetooth_...
device/bluetooth/bluetooth_advertisement_chromeos.cc:54: LOG(WARNING) << "Error
while registering advertisement. error_name = "
LOG(ERROR)?

[rkc, 2015-07-15 20:03:54]

https://codereview.chromium.org/1223173008/diff/1/chromeos/dbus/bluetooth_le_...
File chromeos/dbus/bluetooth_le_advertisement_service_provider.cc (right):

https://codereview.chromium.org/1223173008/diff/1/chromeos/dbus/bluetooth_le_...
chromeos/dbus/bluetooth_le_advertisement_service_provider.cc:261: void
AppendArrayVariantOfStrings(dbus::MessageWriter* dict_writer,
On 2015/07/09 21:03:35, armansito wrote:
> Do you mean AppendVariantArrayOfStrings?

Done.

https://codereview.chromium.org/1223173008/diff/1/chromeos/dbus/bluetooth_le_...
chromeos/dbus/bluetooth_le_advertisement_service_provider.cc:266:
strings_array_variant.OpenArray("s", &strings_array);
On 2015/07/09 21:03:35, armansito wrote:
> You should be able to just call strings_array_variant.AppendArrayOfStrings

Done.

https://codereview.chromium.org/1223173008/diff/1/device/bluetooth/bluetooth_...
File device/bluetooth/bluetooth_advertisement_chromeos.cc (right):

https://codereview.chromium.org/1223173008/diff/1/device/bluetooth/bluetooth_...
device/bluetooth/bluetooth_advertisement_chromeos.cc:54: LOG(WARNING) << "Error
while registering advertisement. error_name = "
On 2015/07/09 21:03:35, armansito wrote:
> LOG(ERROR)?

Done.

[armansito, 2015-07-15 21:24:41]

lgtm

[rkc, 2015-07-15 21:25:26]

The CQ bit was checked by rkc@chromium.org

[commit-bot: I haz the power, 2015-07-15 21:26:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1223173008/20001

[commit-bot: I haz the power, 2015-07-15 22:25:19]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-07-15 22:27:27]

Patchset 2 (id:??) landed as
https://crrev.com/ab23b18bc2937ff716e42805105fe417c829b60c
Cr-Commit-Position: refs/heads/master@{#338933}