Closed
Bug 1165182
Opened 9 years ago
Closed 9 years ago
[bluetooth2] LeScan keeps scanning even after app stopped
Categories
(Firefox OS Graveyard :: Bluetooth, defect)
Tracking
(firefox41 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox41 | --- | fixed |
People
(Reporter: elin, Assigned: jaliu)
References
Details
(Whiteboard: [webbt-api])
Attachments
(1 file, 1 obsolete file)
If I call startLeScan in my app, and then stop my app from WebIDE or task manager without calling stopLeScan first, it will keep scanning until I disable bluetooth.
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → jaliu
Assignee | ||
Updated•9 years ago
|
Status: NEW → ASSIGNED
Updated•9 years ago
|
Whiteboard: [webbt-api]
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8607387 -
Flags: review?(joliu)
Comment 2•9 years ago
|
||
Comment on attachment 8607387 [details] [diff] [review] Stop LE scans when Bluetooth adapter is disconnected from its owner. (v1) Review of attachment 8607387 [details] [diff] [review]: ----------------------------------------------------------------- I'm sorry for the delay here. The patch looks good to me, thanks. ::: dom/bluetooth/bluetooth2/BluetoothAdapter.cpp @@ +340,5 @@ > BluetoothAdapter::DisconnectFromOwner() > { > DOMEventTargetHelper::DisconnectFromOwner(); > > + Cleanup(); nit: remove the empty line before this line.
Attachment #8607387 -
Flags: review?(joliu) → review+
Assignee | ||
Comment 3•9 years ago
|
||
- revise previous patch based on #comment 2 - add reviewer's name to commit message Thank Jocelyn for reviewing the patch.
Attachment #8607387 -
Attachment is obsolete: true
Assignee | ||
Comment 4•9 years ago
|
||
Skip the "Pushing to Try" procedure since Bluetooth API v2 hasn't be covered by tinderbox.
You need to log in
before you can comment on or make changes to this bug.
Description
•