[Bluetooth] Stop discovering before connecting

RESOLVED FIXED

Status

Firefox OS
Gaia::Settings
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: ericchou, Assigned: ericchou)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Bug 873352 has the same idea as this one. Discovery is a heavy operation which would increase the failure rate of pairing or establishing a new profile level connection. Therefore we should do stopDiscovery() every time before we call pair or connect.

Recently I found that 'restore connection' was usually not successful, and the main reason was that bluedroid(bluetooth backend which will be used after FxOS v1.3) would report that RFCOMM connection couldn't be established. Once I tried to put 'stopDiscovery' in doConnect() before calling defaultAdapter.connect, everything went well. So I'd like to raise this issue again otherwise connection error may happen too often.
(Assignee)

Updated

5 years ago
Assignee: nobody → echou
(Assignee)

Updated

5 years ago
See Also: → bug 873352
(Assignee)

Comment 2

5 years ago
Comment on attachment 8337627 [details] [diff] [review]
Link to https://github.com/mozilla-b2g/gaia/pull/14004

* Reset reviewer to Evelyn after talked to Arthur.
Attachment #8337627 - Flags: review?(arthur.chen) → review?(ehung)

Comment 3

5 years ago
Comment on attachment 8337627 [details] [diff] [review]
Link to https://github.com/mozilla-b2g/gaia/pull/14004

looks good to me. Thanks!
Attachment #8337627 - Flags: review?(ehung) → review+

Comment 4

5 years ago
merged into gaia master: https://github.com/mozilla-b2g/gaia/commit/e64c724140bbcd582a5ec5bbe5fd5bf253a444ec
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.