bluetooth: Match real ReadRemoteCharacteristic's behavior

bluetooth: Match real ReadRemoteCharacteristic's behavior

BluetoothRemoteGattCharacteristic::ReadRemoteCharacteristic used to
dispatch an GattCharacteristicValueChanged event once a characteristic was
read. This side effect was removed but our fakes were not changed.

This patch stops calling GattCharacteristicValueChanged after read operations.

BUG=713984
Review-Url: https://codereview.chromium.org/2829853006
Cr-Commit-Position: refs/heads/master@{#466276}
Committed: https://chromium.googlesource.com/chromium/src/+/abe917c952a135427bec96cd62d8b7c174d79734

[ortuno, 2017-04-21 04:20:35]

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

[commit-bot: I haz the power, 2017-04-21 04:21:10]

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

[ortuno, 2017-04-21 04:34:54]

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

[ortuno, 2017-04-21 04:34:55]

scheib: PTAL

[commit-bot: I haz the power, 2017-04-21 05:54:17]

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

[commit-bot: I haz the power, 2017-04-21 05:54:18]

Dry run: This issue passed the CQ dry run.

[scheib, 2017-04-21 06:34:30]

lgtm

[ortuno, 2017-04-21 06:47:10]

The CQ bit was checked by ortuno@chromium.org

[commit-bot: I haz the power, 2017-04-21 06:47:21]

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, 2017-04-21 06:51:56]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1492757230940070, "parent_rev":
"375a20a84a2d9001b4f87fffdb0bb1aa4b4e0862", "commit_rev":
"abe917c952a135427bec96cd62d8b7c174d79734"}

[commit-bot: I haz the power, 2017-04-21 06:52:12]

Description was changed from

==========
bluetooth: Match real ReadRemoteCharacteristic's behavior

BluetoothRemoteGattCharacteristic::ReadRemoteCharacteristic used to
dispatch an GattCharacteristicValueChanged event once a characteristic was
read. This side effect was removed but our fakes were not changed.

This patch stops calling GattCharacteristicValueChanged after read operations.

BUG=713984
==========

to

==========
bluetooth: Match real ReadRemoteCharacteristic's behavior

BluetoothRemoteGattCharacteristic::ReadRemoteCharacteristic used to
dispatch an GattCharacteristicValueChanged event once a characteristic was
read. This side effect was removed but our fakes were not changed.

This patch stops calling GattCharacteristicValueChanged after read operations.

BUG=713984

Review-Url: https://codereview.chromium.org/2829853006
Cr-Commit-Position: refs/heads/master@{#466276}
Committed:
https://chromium.googlesource.com/chromium/src/+/abe917c952a135427bec96cd62d8...
==========

[commit-bot: I haz the power, 2017-04-21 06:52:13]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/abe917c952a135427bec96cd62d8...