Closed Bug 1173266 Opened 7 years ago Closed 7 years ago

[cleanup] Remove unused |isConnected| from bluetooth2


(Firefox OS Graveyard :: Bluetooth, defect)

Gonk (Firefox OS)
Not set


(firefox42 fixed)

FxOS-S4 (07Aug)
Tracking Status
firefox42 --- fixed


(Reporter: tzimmermann, Assigned: ben.tian)




(1 file, 1 obsolete file)

|BluetoothServiceBluedroid::IsConnected| is newer in Bluetooth v1. Bluetooth v2 should adopt this code.

::: dom/bluetooth/bluedroid/BluetoothServiceBluedroid.h
@@ +161,5 @@
>    virtual bool
>    IsConnected(uint16_t aProfileId);
> +#else
> +  virtual void
> +  IsConnected(const uint16_t aServiceUuid,

See also bug 929376 and bug 1172914.
Assignee: nobody → btian
Attachment #8643508 - Flags: review?(joliu)
Attachment #8643508 - Flags: feedback?(jaliu)
Comment on attachment 8643508 [details] [diff] [review]
Patch 1 (v1): Port bug 929376 to bluetooth2

Jamin and I decide not to add this method per offline discussion with Fred. The reason is that System app maintains its own list of connected profiles and doesn't call adapter.isConnected() at all. If there's specific API requirement, gecko can add/redesign API to query whether a profile is connected in the future.

I'll still upload a patch to remove |isConnected| in bluetooth2.
Attachment #8643508 - Attachment is obsolete: true
Attachment #8643508 - Flags: review?(joliu)
Attachment #8643508 - Flags: feedback?(jaliu)
Summary: Use Bluetooth v1 variant for |BluetoothServiceBluedroid::IsConnected| → [cleanup] Remove unused |isConnected| from bluetooth2
Comment on attachment 8643524 [details] [diff] [review]
Patch 2 (v1): Remove unused |isConnected| method from bluetooth2

Review of attachment 8643524 [details] [diff] [review]:

I'm not sure if it makes sense that the System app maintains the list of connected profiles. But since |IsConnected| is unimplemented anyway, we can also remove it.

I also encourage you to completely remove v1 in the near future. :)
Attachment #8643524 - Flags: review?(tzimmermann) → review+
(In reply to Thomas Zimmermann [:tzimmermann] [:tdz] from comment #4)
> I also encourage you to completely remove v1 in the near future. :)

We sure will. v1 is still there since we're working on feature phone features (2.2-based) on m-c branch. v1 can be removed once we move to work on the 2.2-based branch.
It will be nice to have `isConnected` method when we have more than one adapter, so we can remove the connected profiles list and relay the request to the concrete connection status.

In Gaia we also maintain v1 API in master as well, 
please ping me when you plan to remove v1 API, so we can align the plan.
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → FxOS-S4 (07Aug)
You need to log in before you can comment on or make changes to this bug.