arc: bluetooth: Fix LSAN error in fake_bluetooth_instance

arc: bluetooth: Fix LSAN error in fake_bluetooth_instance

fake_bluetooth_instance used raw pointer to store all the
requests to mojo BluetoothInstance. However, this can easisly
caused memory leak unless we are carefully delete those data.

This CL changes it to std::unique_ptr<> to auto destroy those
data when fake_bluetooth_instance is destroyed.

BUG=634898
TEST=Run tests with ASAN and LSAN

Committed: https://crrev.com/dea5e4beeecc2dac9ec84dd30741a24d2214795f
Cr-Commit-Position: refs/heads/master@{#410160}

[puthik_chromium, 2016-08-05 18:50:18]

puthik@chromium.org changed reviewers:
+ anthonyvd@chromium.org

[puthik_chromium, 2016-08-05 19:02:09]

The CQ bit was checked by puthik@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-05 19:02:44]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Reilly Grant (use Gerrit), 2016-08-05 19:30:52]

reillyg@chromium.org changed reviewers:
+ reillyg@chromium.org

[Reilly Grant (use Gerrit), 2016-08-05 19:30:53]

lgtm

Please land this patch as soon as possible. The tree should not be left red.

[puthik_chromium, 2016-08-05 20:22:51]

puthik@chromium.org changed reviewers:
+ elijahtaylor@chromium.org

[puthik_chromium, 2016-08-05 20:22:52]

+Owner
Look like lhchavez@ is OoO today.

[puthik_chromium, 2016-08-05 20:25:57]

The CQ bit was unchecked by puthik@chromium.org

[elijahtaylor1, 2016-08-05 20:31:27]

lgtm

[puthik_chromium, 2016-08-05 20:31:56]

The CQ bit was checked by puthik@chromium.org

[commit-bot: I haz the power, 2016-08-05 20:33:24]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-05 21:09:08]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-05 21:11:11]

Description was changed from

==========
arc: bluetooth: Fix LSAN error in fake_bluetooth_instance

fake_bluetooth_instance used raw pointer to store all the
requests to mojo BluetoothInstance. However, this can easisly
caused memory leak unless we are carefully delete those data.

This CL changes it to std::unique_ptr<> to auto destroy those
data when fake_bluetooth_instance is destroyed.

BUG=634898
TEST=Run tests with ASAN and LSAN
==========

to

==========
arc: bluetooth: Fix LSAN error in fake_bluetooth_instance

fake_bluetooth_instance used raw pointer to store all the
requests to mojo BluetoothInstance. However, this can easisly
caused memory leak unless we are carefully delete those data.

This CL changes it to std::unique_ptr<> to auto destroy those
data when fake_bluetooth_instance is destroyed.

BUG=634898
TEST=Run tests with ASAN and LSAN

Committed: https://crrev.com/dea5e4beeecc2dac9ec84dd30741a24d2214795f
Cr-Commit-Position: refs/heads/master@{#410160}
==========

[commit-bot: I haz the power, 2016-08-05 21:11:12]

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