bluetooth: Improve error message for retrieving Descriptors when disconnecting

bluetooth: Improve error message for retrieving Descriptors when disconnecting

Changes characteristic.getDescriptor(s) to return a more actionable error.


BUG=684445
Review-Url: https://codereview.chromium.org/2728813002
Cr-Commit-Position: refs/heads/master@{#455929}
Committed: https://chromium.googlesource.com/chromium/src/+/258758c687d31cc18e1f9f0395a8714df0187202

[ortuno, 2017-03-01 23:50:27]

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

[ortuno, 2017-03-01 23:50:28]

scheib: PTAL

[ortuno, 2017-03-01 23:50:35]

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

[commit-bot: I haz the power, 2017-03-01 23:51:31]

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, 2017-03-01 23:57:28]

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

[commit-bot: I haz the power, 2017-03-01 23:57:28]

Dry run: Try jobs failed on following builders:
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[scheib, 2017-03-04 19:17:57]

lgtm, but maybe change your git cl upload --similarity 10 to help reviews like
this.

[ortuno, 2017-03-07 04:14:44]

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

[commit-bot: I haz the power, 2017-03-07 04:15:36]

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, 2017-03-07 04:21:13]

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

[commit-bot: I haz the power, 2017-03-07 04:21:14]

Dry run: Try jobs failed on following builders:
  chromeos_amd64-generic_chromium_compile_only_ng on
master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[ortuno, 2017-03-09 02:52:31]

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

[commit-bot: I haz the power, 2017-03-09 02:52:54]

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, 2017-03-09 03:54:25]

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

[commit-bot: I haz the power, 2017-03-09 03:54: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, 2017-03-09 22:27:06]

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

[commit-bot: I haz the power, 2017-03-09 22:27:49]

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, 2017-03-10 00:14:48]

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

[commit-bot: I haz the power, 2017-03-10 00:14:49]

Dry run: This issue passed the CQ dry run.

[ortuno, 2017-03-10 00:39:58]

The CQ bit was checked by ortuno@chromium.org

[ortuno, 2017-03-10 00:39:58]

The patchset sent to the CQ was uploaded after l-g-t-m from scheib@chromium.org
Link to the patchset: https://codereview.chromium.org/2728813002/#ps80001
(title: "Fix conflict error")

[commit-bot: I haz the power, 2017-03-10 00:40:48]

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-03-10 00:48:07]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1489106398181020,
"parent_rev": "347e05c9fcda35ab2fd31cd7e7b9d89bc10aed26", "commit_rev":
"258758c687d31cc18e1f9f0395a8714df0187202"}

[commit-bot: I haz the power, 2017-03-10 00:48:50]

Description was changed from

==========
bluetooth: Improve error message for retrieving Descriptors when disconnecting

Changes characteristic.getDescriptor(s) to return a more actionable error.


BUG=684445
==========

to

==========
bluetooth: Improve error message for retrieving Descriptors when disconnecting

Changes characteristic.getDescriptor(s) to return a more actionable error.


BUG=684445

Review-Url: https://codereview.chromium.org/2728813002
Cr-Commit-Position: refs/heads/master@{#455929}
Committed:
https://chromium.googlesource.com/chromium/src/+/258758c687d31cc18e1f9f0395a8...
==========

[commit-bot: I haz the power, 2017-03-10 00:48:51]

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/258758c687d31cc18e1f9f0395a8...