bluetooth: Make BluetoothRemoteGATTCharacteristic Persistent

bluetooth: Make BluetoothRemoteGATTCharacteristic Persistent

Since it was WeakPersistent the characteristic was getting garbage
collected which lead to a crash when the response arrived. Also adds tests for
all the relevant functions.

BUG=659182
Committed: https://crrev.com/35d4f33d891b86e982d9f010285fc71937de3bf2
Cr-Commit-Position: refs/heads/master@{#429509}

[ortuno, 2016-11-02 05:13:32]

Description was changed from

==========
bluetooth: Make BluetoothRemoteGATTCharacteristic Persistent

Since it was WeakPersistent the characteristic was getting garbage
collected which lead to a crash when the response arrived.

BUG=659182
==========

to

==========
bluetooth: Make BluetoothRemoteGATTCharacteristic Persistent

Since it was WeakPersistent the characteristic was getting garbage
collected which lead to a crash when the response arrived. Also adds tests for
all the relevant functions.

BUG=659182
==========

[ortuno, 2016-11-02 05:13:40]

ortuno@chromium.org changed reviewers:
+ scheib@chromium.org

[ortuno, 2016-11-02 05:13:46]

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

[commit-bot: I haz the power, 2016-11-02 05:14:06]

Dry run: 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-11-02 06:46:25]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-02 06:46:26]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[ortuno, 2016-11-02 22:46:55]

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

[commit-bot: I haz the power, 2016-11-02 22:47:59]

Dry run: 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-11-03 00:32:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-03 00:32:36]

Dry run: This issue passed the CQ dry run.

[scheib, 2016-11-03 01:20:02]

LGTM

[ortuno, 2016-11-03 01:41:38]

The CQ bit was checked by ortuno@chromium.org

[ortuno, 2016-11-03 01:41:38]

Thanks!

[commit-bot: I haz the power, 2016-11-03 01:41:53]

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-11-03 01:58:54]

Committed patchset #6 (id:100001)

[commit-bot: I haz the power, 2016-11-03 02:03:00]

Description was changed from

==========
bluetooth: Make BluetoothRemoteGATTCharacteristic Persistent

Since it was WeakPersistent the characteristic was getting garbage
collected which lead to a crash when the response arrived. Also adds tests for
all the relevant functions.

BUG=659182
==========

to

==========
bluetooth: Make BluetoothRemoteGATTCharacteristic Persistent

Since it was WeakPersistent the characteristic was getting garbage
collected which lead to a crash when the response arrived. Also adds tests for
all the relevant functions.

BUG=659182

Committed: https://crrev.com/35d4f33d891b86e982d9f010285fc71937de3bf2
Cr-Commit-Position: refs/heads/master@{#429509}
==========

[commit-bot: I haz the power, 2016-11-03 02:03:01]

Patchset 6 (id:??) landed as
https://crrev.com/35d4f33d891b86e982d9f010285fc71937de3bf2
Cr-Commit-Position: refs/heads/master@{#429509}