Closed
Bug 1230066
Opened 9 years ago
Closed 9 years ago
Add missing STATUS_AUTH_REJECTED into enum BluetoothStatus
Categories
(Firefox OS Graveyard :: Bluetooth, defect)
Tracking
(firefox45 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox45 | --- | fixed |
People
(Reporter: ben.tian, Assigned: ben.tian)
Details
Attachments
(1 file)
1.62 KB,
patch
|
tzimmermann
:
review+
|
Details | Diff | Splinter Review |
Per bug 1229697 comment 3, we should add STATUS_AUTH_REJECTED into BluetoothDaemonHelpers [1] as BlueZ protocol has it [2]. The status code is received on device when remote device rejects pincode pairing.
[1] https://dxr.mozilla.org/mozilla-central/source/dom/bluetooth/bluedroid/BluetoothDaemonHelpers.cpp
[2] https://git.kernel.org/cgit/bluetooth/bluez.git/tree/android/hal-ipc-api.txt#n136
Comment 1•9 years ago
|
||
Oh, thanks! It looks like this was added in 5.0. Shall I take this bug?
Assignee | ||
Comment 2•9 years ago
|
||
Assignee: nobody → btian
Attachment #8695219 -
Flags: review?(tzimmermann)
Comment 3•9 years ago
|
||
Comment on attachment 8695219 [details] [diff] [review]
Patch 1 (v1): Add missing STATUS_AUTH_REJECTED into enum BluetoothStatus
Review of attachment 8695219 [details] [diff] [review]:
-----------------------------------------------------------------
Thanks for this!
Attachment #8695219 -
Flags: review?(tzimmermann) → review+
Comment 5•9 years ago
|
||
bugherder |
You need to log in
before you can comment on or make changes to this bug.
Description
•