Closed
Bug 1103872
Opened 10 years ago
Closed 10 years ago
[Bluetooth] Port Bug 1091588 to bluetooth2
Categories
(Firefox OS Graveyard :: Bluetooth, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: tzimmermann, Assigned: tzimmermann)
References
Details
Attachments
(4 files, 2 obsolete files)
24.50 KB,
patch
|
ben.tian
:
review+
|
Details | Diff | Splinter Review |
41.48 KB,
patch
|
ben.tian
:
review+
|
Details | Diff | Splinter Review |
6.50 KB,
patch
|
ben.tian
:
review+
|
Details | Diff | Splinter Review |
11.79 KB,
patch
|
tzimmermann
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8528379 -
Flags: review?(btian)
Assignee | ||
Comment 2•10 years ago
|
||
Attachment #8528380 -
Flags: review?(btian)
Assignee | ||
Comment 3•10 years ago
|
||
Attachment #8528381 -
Flags: review?(btian)
Assignee | ||
Comment 4•10 years ago
|
||
Attachment #8528382 -
Flags: review?(btian)
Comment 5•10 years ago
|
||
Comment on attachment 8528379 [details] [diff] [review]
[01] Bug 1103872: Add helpers for Bluetooth daemon Handsfree support
Review of attachment 8528379 [details] [diff] [review]:
-----------------------------------------------------------------
LGTM.
Attachment #8528379 -
Flags: review?(btian) → review+
Comment 6•10 years ago
|
||
Comment on attachment 8528380 [details] [diff] [review]
[02] Bug 1103872: Add Handsfree module for Bluetooth daemon
Review of attachment 8528380 [details] [diff] [review]:
-----------------------------------------------------------------
LGTM.
Attachment #8528380 -
Flags: review?(btian) → review+
Comment 7•10 years ago
|
||
Comment on attachment 8528381 [details] [diff] [review]
[03] Bug 1103872: Add Handsfree interface for Bluetooth daemon
Review of attachment 8528381 [details] [diff] [review]:
-----------------------------------------------------------------
r=me with typo addressed.
::: dom/bluetooth2/bluedroid/BluetoothDaemonHandsfreeInterface.h
@@ +331,5 @@
> +
> + enum {
> + MODE_HEADSET = 0x00,
> + MODE_NARROWBAND_SPEECH = 0x01,
> + MODE_NARRAWBAND_WIDEBAND_SPEECH = 0x02
typo: NARR'O'WBAND_WIDEBAND
Attachment #8528381 -
Flags: review?(btian) → review+
Comment 8•10 years ago
|
||
Comment on attachment 8528382 [details] [diff] [review]
[04] Bug 1103872: Support Handsfree profile when using Bluetooth daemon
Review of attachment 8528382 [details] [diff] [review]:
-----------------------------------------------------------------
LGTM.
Attachment #8528382 -
Flags: review?(btian) → review+
Assignee | ||
Comment 9•10 years ago
|
||
Changes since v1:
- fixed typo in enum constant
Attachment #8528382 -
Attachment is obsolete: true
Attachment #8530228 -
Flags: review+
Assignee | ||
Updated•10 years ago
|
Attachment #8530228 -
Attachment is obsolete: true
Assignee | ||
Comment 10•10 years ago
|
||
Comment on attachment 8528382 [details] [diff] [review]
[04] Bug 1103872: Support Handsfree profile when using Bluetooth daemon
Restoring correct patch [04](v1)
Attachment #8528382 -
Attachment is obsolete: false
Assignee | ||
Comment 11•10 years ago
|
||
Changes since v1:
- fixed typo in enum constant
Attachment #8528381 -
Attachment is obsolete: true
Attachment #8530230 -
Flags: review+
Assignee | ||
Comment 12•10 years ago
|
||
Thanks!
https://hg.mozilla.org/integration/b2g-inbound/rev/cb9fb4a16519
https://hg.mozilla.org/integration/b2g-inbound/rev/b9b07db1e846
https://hg.mozilla.org/integration/b2g-inbound/rev/6708343dd7ee
https://hg.mozilla.org/integration/b2g-inbound/rev/eb2a8052241b
https://treeherder.mozilla.org/ui/#/jobs?repo=b2g-inbound&revision=eb2a8052241b
Comment 13•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/cb9fb4a16519
https://hg.mozilla.org/mozilla-central/rev/b9b07db1e846
https://hg.mozilla.org/mozilla-central/rev/6708343dd7ee
https://hg.mozilla.org/mozilla-central/rev/eb2a8052241b
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•