Closed Bug 757713 Opened 12 years ago Closed 12 years ago

B2G Bluetooth: Get adapter path

Categories

(Core :: DOM: Device Interfaces, defect)

x86_64
Gonk (Firefox OS)
defect
Not set
normal

Tracking

()

RESOLVED DUPLICATE of bug 744349

People

(Reporter: echou, Unassigned)

Details

Attachments

(1 file)

Low level function to get default BluetoothAdapter path.
The adapter path is a string representing a BluetoothAdapter object. It's also one of parameters of all BluetoothAdapter related dbus functions.
Attachment #626313 - Flags: feedback?(kyle)
Comment on attachment 626313 [details] [diff] [review]
WIP: still waiting for other patches checking in, feedback first

Review of attachment 626313 [details] [diff] [review]:
-----------------------------------------------------------------

GetAdapterPath, which I'm assuming will actually be calling dbus's DefaultAdapter, is already part of the DBus Signal Handler patch at bug 744349 (check out the WIP I have over there already). Main reason I did that is because I needed an example of signal hookups and that seemed best. Also, the only thing that should need to call DefaultAdapter is BluetoothManager, since it will establish the Adapter objects, and for now I think we'll only have one object. BluetoothManager is a RawDBusConnection so it should have its own connection to the system bus already.

That said, we can break out the calling of the GetDefaultAdapterPath code there and land it first here, as signals requires the dbus node name, but not vice versa.
Attachment #626313 - Flags: feedback?(kyle)
Component: DOM → DOM: Device Interfaces
QA Contact: general → device-interfaces
OK, I'll mark this issue as a of bug 744349.
OK, I'll mark this issue as a duplicate of bug 744349. I saw Kyle's WIP patch and I think that's what we want now.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: