bluetooth: android: Implement Descriptor Write as needed for notifications.

bluetooth: android: Implement Descriptor Write as needed for notifications.

Implements the ability to write to descriptor values in preparation
for [following patch] that will use descriptors for enabling notifications.

Testing is deferred to [following patch], as this code isn't exposed
to public API until that patch.

[following patch] https://codereview.chromium.org/1502833002/
BUG=566533
Committed: https://crrev.com/98cb300cb67c773598bd01184b0ffce8537004b1
Cr-Commit-Position: refs/heads/master@{#369105}

[scheib, 2016-01-13 00:53:30]

Description was changed from

==========
bluetooth: android: Implement Descriptor Write as needed for notifications.

Implements the ability to write to descriptor values in preparation
for following patch that will use descriptors for enabling notifications.

BUG=566533
==========

to

==========
bluetooth: android: Implement Descriptor Write as needed for notifications.

Implements the ability to write to descriptor values in preparation
for [following patch] that will use descriptors for enabling notifications.

Use of this code isn't currently exposed to C++, testing it is done implicitly
through the notification use in  [following patch].

[following patch] https://codereview.chromium.org/1502833002/
BUG=566533
==========

[scheib, 2016-01-13 00:54:32]

Description was changed from

==========
bluetooth: android: Implement Descriptor Write as needed for notifications.

Implements the ability to write to descriptor values in preparation
for [following patch] that will use descriptors for enabling notifications.

Use of this code isn't currently exposed to C++, testing it is done implicitly
through the notification use in  [following patch].

[following patch] https://codereview.chromium.org/1502833002/
BUG=566533
==========

to

==========
bluetooth: android: Implement Descriptor Write as needed for notifications.

Implements the ability to write to descriptor values in preparation
for [following patch] that will use descriptors for enabling notifications.

Testing is deferred to [following patch], as this code isn't exposed
to public API until that patch.

[following patch] https://codereview.chromium.org/1502833002/
BUG=566533
==========

[scheib, 2016-01-13 00:57:09]

Patchset #1 (id:1) has been deleted

[scheib, 2016-01-13 00:57:25]

Patchset #1 (id:20001) has been deleted

[Jeffrey Yasskin, 2016-01-13 01:52:34]

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

[Jeffrey Yasskin, 2016-01-13 01:52:34]

lgtm

https://codereview.chromium.org/1577623002/diff/40001/device/bluetooth/test/a...
File
device/bluetooth/test/android/java/src/org/chromium/device/bluetooth/Fakes.java
(right):

https://codereview.chromium.org/1577623002/diff/40001/device/bluetooth/test/a...
device/bluetooth/test/android/java/src/org/chromium/device/bluetooth/Fakes.java:603:
// Cause subsequent value write of a descriptor to fail synchronously.
I guess we don't need the 'read' version yet, because enabling
notifications/indications only needs a write.

[scheib, 2016-01-13 02:01:26]

https://codereview.chromium.org/1577623002/diff/40001/device/bluetooth/test/a...
File
device/bluetooth/test/android/java/src/org/chromium/device/bluetooth/Fakes.java
(right):

https://codereview.chromium.org/1577623002/diff/40001/device/bluetooth/test/a...
device/bluetooth/test/android/java/src/org/chromium/device/bluetooth/Fakes.java:603:
// Cause subsequent value write of a descriptor to fail synchronously.
On 2016/01/13 01:52:34, Jeffrey Yasskin wrote:
> I guess we don't need the 'read' version yet, because enabling
> notifications/indications only needs a write.

Affirmative.

[scheib, 2016-01-13 02:40:21]

The CQ bit was checked by scheib@chromium.org

[commit-bot: I haz the power, 2016-01-13 02:41:26]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1577623002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1577623002/40001

[commit-bot: I haz the power, 2016-01-13 03:52:25]

Description was changed from

==========
bluetooth: android: Implement Descriptor Write as needed for notifications.

Implements the ability to write to descriptor values in preparation
for [following patch] that will use descriptors for enabling notifications.

Testing is deferred to [following patch], as this code isn't exposed
to public API until that patch.

[following patch] https://codereview.chromium.org/1502833002/
BUG=566533
==========

to

==========
bluetooth: android: Implement Descriptor Write as needed for notifications.

Implements the ability to write to descriptor values in preparation
for [following patch] that will use descriptors for enabling notifications.

Testing is deferred to [following patch], as this code isn't exposed
to public API until that patch.

[following patch] https://codereview.chromium.org/1502833002/
BUG=566533
==========

[commit-bot: I haz the power, 2016-01-13 03:52:25]

Committed patchset #1 (id:40001)

[commit-bot: I haz the power, 2016-01-13 03:53:40]

Description was changed from

==========
bluetooth: android: Implement Descriptor Write as needed for notifications.

Implements the ability to write to descriptor values in preparation
for [following patch] that will use descriptors for enabling notifications.

Testing is deferred to [following patch], as this code isn't exposed
to public API until that patch.

[following patch] https://codereview.chromium.org/1502833002/
BUG=566533
==========

to

==========
bluetooth: android: Implement Descriptor Write as needed for notifications.

Implements the ability to write to descriptor values in preparation
for [following patch] that will use descriptors for enabling notifications.

Testing is deferred to [following patch], as this code isn't exposed
to public API until that patch.

[following patch] https://codereview.chromium.org/1502833002/
BUG=566533

Committed: https://crrev.com/98cb300cb67c773598bd01184b0ffce8537004b1
Cr-Commit-Position: refs/heads/master@{#369105}
==========

[commit-bot: I haz the power, 2016-01-13 03:53:40]

Patchset 1 (id:??) landed as
https://crrev.com/98cb300cb67c773598bd01184b0ffce8537004b1
Cr-Commit-Position: refs/heads/master@{#369105}