Closed
Bug 802288
Opened 12 years ago
Closed 12 years ago
[b2g-bluetooth] BluetoothOppManager does not destruct correctly on shutdown
Categories
(Core :: DOM: Device Interfaces, defect)
Tracking
()
People
(Reporter: qdot, Assigned: qdot)
References
Details
Attachments
(1 file)
3.48 KB,
patch
|
echou
:
review+
akeybl
:
approval-mozilla-aurora+
akeybl
:
approval-mozilla-b2g18+
|
Details | Diff | Splinter Review |
Not that we ever shut the phone down this way, but...
On b2g desktop, we get XPCOM errors on shutdown due to BluetoothOppManager not being destructed at time of xpcom-shutdown event. We just need the same listeners that BluetoothHfp/ScoManager have already.
Assignee | ||
Comment 1•12 years ago
|
||
Failing shutdown like this shouldn't cause crashes. Marking blocking+.
Assignee: nobody → kyle
blocking-basecamp: --- → ?
Comment 2•12 years ago
|
||
We don't think we'll be doing shutdowns in v1 and instead just killing the process.
blocking-basecamp: ? → -
Assignee | ||
Comment 3•12 years ago
|
||
This is pretty much copy/paste from HfpManager, and I have little to no idea what happens in OppManager, so let me know if bringing things down this way is bad (i.e. if we need to "gracefully" disconnect or something on shutdown). But, while not sending a file, this clears up our XPCOM bitching on shutdown.
Attachment #674925 -
Flags: review?(echou)
Comment 4•12 years ago
|
||
Comment on attachment 674925 [details] [diff] [review]
Patch 1 (v1) - Make BluetoothOppManager shutdown at xpcom-shutdown
Review of attachment 674925 [details] [diff] [review]:
-----------------------------------------------------------------
Even if we are sending or receiving data via OPP, CloseSocket() should be a way to tell remote side that we are going to close this session.
Attachment #674925 -
Flags: review?(echou) → review+
Assignee | ||
Comment 5•12 years ago
|
||
Comment 6•12 years ago
|
||
Status: NEW → RESOLVED
Closed: 12 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
Comment 7•12 years ago
|
||
Comment on attachment 674925 [details] [diff] [review]
Patch 1 (v1) - Make BluetoothOppManager shutdown at xpcom-shutdown
Since patches in Bug 819832(bb+) and Bug 817930(bb+, P1, C3) have conflicts on mozilla-beta and mozilla-aurora, ask for approval of mozilla-beta and mozilla-aurora.
[Approval Request Comment]
Bug caused by (feature/regressing bug #): Bug 792823 (BluetoothOppManager)
User impact if declined: BluetoothOppManager won't be destroyed properly when we shut down our phone
Testing completed (on m-c, etc.): on m-c
Risk to taking this patch (and alternatives if risky): fairly low, just make BluetoothOppManager as an observer of xpcom-shutdown, and reset everything when it is notified.
String or UUID changes made by this patch: none
Attachment #674925 -
Flags: approval-mozilla-beta?
Attachment #674925 -
Flags: approval-mozilla-aurora?
Comment 8•12 years ago
|
||
Comment on attachment 674925 [details] [diff] [review]
Patch 1 (v1) - Make BluetoothOppManager shutdown at xpcom-shutdown
please make sure to land on mozilla-b2g18, and not mozilla-beta.
Attachment #674925 -
Flags: approval-mozilla-beta?
Attachment #674925 -
Flags: approval-mozilla-b2g18+
Attachment #674925 -
Flags: approval-mozilla-aurora?
Attachment #674925 -
Flags: approval-mozilla-aurora+
Comment 9•12 years ago
|
||
https://hg.mozilla.org/releases/mozilla-b2g18/rev/3c3b15f19b9d
Note that this landed on m-c when it was still Mozilla19 and is therefore already on Aurora. Also, this had some conflicts that I had to work around, so you should probably take a look to make sure you're happy with the end result.
status-firefox19:
--- → fixed
Whiteboard: [status-b2g18:fixed]
Assignee | ||
Comment 10•12 years ago
|
||
Looks ok to me.
Comment 11•12 years ago
|
||
Thanks for your help, Ryan. I've confirmed the changeset of 3c3b15f19b9d and it's perfect :)
Updated•12 years ago
|
status-b2g18:
--- → fixed
Whiteboard: [status-b2g18:fixed]
You need to log in
before you can comment on or make changes to this bug.
Description
•