Closed
Bug 812404
Opened 12 years ago
Closed 12 years ago
[b2g-bluetooth] Crash when connect with a bluetooth headset right after pairing completed
Categories
(Core :: DOM: Device Interfaces, defect)
Tracking
()
People
(Reporter: gyeh, Assigned: gyeh)
References
Details
Attachments
(1 file, 2 obsolete files)
(deleted),
patch
|
Details | Diff | Splinter Review |
Repro:
- Enable bluetooth
- Pair with a bluetooth headset
- Connect with it immediately
Expected:
- Connect success
Actual:
- Crashed
In the implementation of settings app, after pairing with a bluetooth headset, we'll automatically connect with it after 5 seconds. However, user might try to connect with the headset at this time, resulting in b2g crashed, so we need to handle this situation.
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #682305 -
Flags: review?(echou)
Comment 2•12 years ago
|
||
Comment on attachment 682305 [details] [diff] [review]
Patch 1(v1): Return when connection status is connecting
Review of attachment 682305 [details] [diff] [review]:
-----------------------------------------------------------------
r+ with nits addressed.
::: dom/bluetooth/ipc/BluetoothServiceChildProcess.cpp
@@ +301,2 @@
> SendRequest(aRunnable,
> ConnectRequest(nsString(aDeviceAddress),
nit: an additional blank
::: dom/bluetooth/linux/BluetoothDBusService.cpp
@@ +2314,1 @@
> NS_WARNING("Unknow Profile");
nit: a typo here, please help to fix. :)
Attachment #682305 -
Flags: review?(echou) → review+
Assignee | ||
Comment 3•12 years ago
|
||
try:
https://tbpl.mozilla.org/?tree=Try&rev=c3d3ff55d5f7
https://tbpl.mozilla.org/?tree=Try&rev=0dd3d03c302d
Attachment #682305 -
Attachment is obsolete: true
Assignee | ||
Comment 4•12 years ago
|
||
https://tbpl.mozilla.org/?tree=Try&rev=4edca193359f
https://tbpl.mozilla.org/?tree=Try&rev=4410e7a3b87b
Attachment #682334 -
Attachment is obsolete: true
Assignee | ||
Comment 5•12 years ago
|
||
Comment 6•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
Updated•12 years ago
|
blocking-basecamp: ? → +
Comment 7•12 years ago
|
||
status-firefox18:
--- → fixed
status-firefox19:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•