Closed
Bug 933113
Opened 11 years ago
Closed 11 years ago
Needs a new mechanism for pairing with another Bluetooth device when apps have Bluetooth device address
Categories
(Firefox OS Graveyard :: Bluetooth, defect)
Tracking
(blocking-b2g:1.3+, firefox28 fixed)
Tracking | Status | |
---|---|---|
firefox28 | --- | fixed |
People
(Reporter: echou, Assigned: echou)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
gyeh
:
review+
mrbkap
:
superreview+
|
Details | Diff | Splinter Review |
In the discussion of NFC workweek, we need a new mechanism or a new API to pair with remote devices by their Bluetooth device address. A BluetoothDevice object is necessary for pairing, but Gaia apps aren't able to get one except through event DeviceFound or function GetPairedDevices().
Assignee | ||
Comment 1•11 years ago
|
||
Per offline discussion with Gina and Ben, we decided to change the parameter of Pair/Unpair, replacing BluetoothDevice with Bluetooth device address(DOMString). Will start to work on it tomorrow.
Assignee | ||
Comment 2•11 years ago
|
||
NFC handover is a mechanism for two devices to exchange their Bluetooth device address with each other. Then one device would be initialize the Bluetooth pairing process with received Bluetooth device address. However currently we need a BluetoothDevice object for Pairing/Unpairing, but actually the address of the Bluetooth device object is the only thing Gecko requires.
sr? mrbkap since the patch changes the Pair/Unpair API.
Settings app is the only one using these two APIs so the revision must be necessary. The followup is bug 938472.
Assignee: nobody → echou
Attachment #832010 -
Flags: superreview?(mrbkap)
Attachment #832010 -
Flags: review?(gyeh)
Assignee | ||
Comment 3•11 years ago
|
||
Nominate as 1.3+ since it blocks 1.3+ bug 903305.
blocking-b2g: --- → 1.3?
Assignee | ||
Comment 4•11 years ago
|
||
* Patch updated because I forgot removing comments.
Attachment #832010 -
Attachment is obsolete: true
Attachment #832010 -
Flags: superreview?(mrbkap)
Attachment #832010 -
Flags: review?(gyeh)
Attachment #832050 -
Flags: superreview?(mrbkap)
Attachment #832050 -
Flags: review?(gyeh)
Comment 5•11 years ago
|
||
(In reply to Eric Chou [:ericchou] [:echou] from comment #3)
> Nominate as 1.3+ since it blocks 1.3+ bug 903305.
It is 1.3+.
blocking-b2g: 1.3? → 1.3+
Comment 7•11 years ago
|
||
Comment on attachment 832050 [details] [diff] [review]
patch 1: v2: modify Pair/Unpair interface for Gaia to be able to Pair/Unpair with BD_Address
Review of attachment 832050 [details] [diff] [review]:
-----------------------------------------------------------------
I think it should be fine to change the parameter of Pair/Unpari from a BluetoothDevice to device address.
Attachment #832050 -
Flags: review?(gyeh) → review+
Updated•11 years ago
|
Attachment #832050 -
Flags: superreview?(mrbkap) → superreview+
Assignee | ||
Comment 8•11 years ago
|
||
Assignee | ||
Comment 9•11 years ago
|
||
https://hg.mozilla.org/integration/b2g-inbound/rev/b14b67c82e4d
Thank you both. :)
Comment 10•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
status-firefox28:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•