Closed
Bug 1168446
Opened 9 years ago
Closed 9 years ago
Cleanup interface of |BluetoothSocket|
Categories
(Firefox OS Graveyard :: Bluetooth, defect)
Tracking
(firefox41 fixed)
RESOLVED
FIXED
2.2 S13 (29may)
Tracking | Status | |
---|---|---|
firefox41 | --- | fixed |
People
(Reporter: tzimmermann, Assigned: tzimmermann)
References
Details
Attachments
(4 files)
8.24 KB,
patch
|
shawnjohnjr
:
review+
|
Details | Diff | Splinter Review |
3.14 KB,
patch
|
shawnjohnjr
:
review+
|
Details | Diff | Splinter Review |
6.08 KB,
patch
|
shawnjohnjr
:
review+
|
Details | Diff | Splinter Review |
27.89 KB,
patch
|
shawnjohnjr
:
review+
|
Details | Diff | Splinter Review |
With the changes from bug 1161020, we can clean up the interface of |BluetoothSocket| to be more like the interfaces in ipc/unixsocket. Some methods can also be removed.
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8611105 -
Flags: review?(shuang)
Assignee | ||
Comment 2•9 years ago
|
||
Attachment #8611106 -
Flags: review?(shuang)
Assignee | ||
Comment 3•9 years ago
|
||
Attachment #8611107 -
Flags: review?(shuang)
Assignee | ||
Comment 4•9 years ago
|
||
Attachment #8611109 -
Flags: review?(shuang)
Assignee | ||
Comment 5•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=5e0efdc09998
Assignee | ||
Comment 6•9 years ago
|
||
There was a problem with the build infrastructure. Trying again https://treeherder.mozilla.org/#/jobs?repo=try&revision=a0c65af943db
Attachment #8611105 -
Flags: review?(shuang) → review+
Attachment #8611106 -
Flags: review?(shuang) → review+
Attachment #8611107 -
Flags: review?(shuang) → review+
Comment on attachment 8611109 [details] [diff] [review] [04] Bug 1168446: Don't store connection parameters in |BluetoothSocket| Review of attachment 8611109 [details] [diff] [review]: ----------------------------------------------------------------- It looks good to me for setting auth/encrypt parameters in connect/listen.
Attachment #8611109 -
Flags: review?(shuang) → review+
Assignee | ||
Comment 8•9 years ago
|
||
Thanks Shawn! https://treeherder.mozilla.org/#/jobs?repo=b2g-inbound&revision=37c9f1961a9d
https://hg.mozilla.org/integration/b2g-inbound/rev/543d15b14575 https://hg.mozilla.org/integration/b2g-inbound/rev/fdbfea9d6c0c https://hg.mozilla.org/integration/b2g-inbound/rev/b8289e7701c9
https://hg.mozilla.org/mozilla-central/rev/543d15b14575 https://hg.mozilla.org/mozilla-central/rev/fdbfea9d6c0c https://hg.mozilla.org/mozilla-central/rev/b8289e7701c9 https://hg.mozilla.org/mozilla-central/rev/37c9f1961a9d
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox41:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 2.2 S13 (29may)
You need to log in
before you can comment on or make changes to this bug.
Description
•