Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1426)

Issue 379943003: Retrieve "connected" property from BLE devices. (Closed)

Created:
6 years, 5 months ago by rpaquay
Modified:
6 years, 5 months ago
Reviewers:
xiyuan
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@bluetooth_unittests
Project:
chromium
Visibility:
Public.

Description

Retrieve "connected" property from BLE devices. We have to use the SetupDIGetDeviceProperty API to retrieve this specific property, as the "connected" status is something common to many PnP devices. BUG=388016 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=282212 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=282633

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+144 lines, -8 lines) Patch
M device/bluetooth/bluetooth_low_energy_win.h View 2 chunks +28 lines, -2 lines 0 comments Download
M device/bluetooth/bluetooth_low_energy_win.cc View 12 chunks +104 lines, -6 lines 0 comments Download
M device/bluetooth/bluetooth_low_energy_win_unittest.cc View 1 chunk +11 lines, -0 lines 0 comments Download
M device/bluetooth/bluetooth_task_manager_win.cc View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 10 (0 generated)
rpaquay
6 years, 5 months ago (2014-07-09 20:34:12 UTC) #1
xiyuan
lgtm
6 years, 5 months ago (2014-07-09 20:42:30 UTC) #2
rpaquay
The CQ bit was checked by rpaquay@chromium.org
6 years, 5 months ago (2014-07-09 21:57:55 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rpaquay@chromium.org/379943003/1
6 years, 5 months ago (2014-07-09 21:58:57 UTC) #4
commit-bot: I haz the power
Change committed as 282212
6 years, 5 months ago (2014-07-10 01:36:49 UTC) #5
horo
A revert of this CL has been created in https://codereview.chromium.org/380173002/ by horo@chromium.org. The reason for ...
6 years, 5 months ago (2014-07-10 09:32:29 UTC) #6
horo
On 2014/07/10 09:32:29, horo wrote: > A revert of this CL has been created in ...
6 years, 5 months ago (2014-07-10 09:34:51 UTC) #7
rpaquay
The CQ bit was checked by rpaquay@chromium.org
6 years, 5 months ago (2014-07-11 16:19:12 UTC) #8
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rpaquay@chromium.org/379943003/1
6 years, 5 months ago (2014-07-11 16:19:46 UTC) #9
commit-bot: I haz the power
6 years, 5 months ago (2014-07-11 16:21:09 UTC) #10
Message was sent while issue was closed.
Change committed as 282633

Powered by Google App Engine
This is Rietveld 408576698