Closed Bug 863620 Opened 8 years ago Closed 8 years ago

follow-up to bug 848414

Categories

(Firefox OS Graveyard :: Bluetooth, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(blocking-b2g:tef+, firefox21 wontfix, firefox22 wontfix, firefox23 fixed, b2g18 fixed, b2g18-v1.0.0 wontfix, b2g18-v1.0.1 fixed)

RESOLVED FIXED
blocking-b2g tef+
Tracking Status
firefox21 --- wontfix
firefox22 --- wontfix
firefox23 --- fixed
b2g18 --- fixed
b2g18-v1.0.0 --- wontfix
b2g18-v1.0.1 --- fixed

People

(Reporter: echou, Assigned: gyeh)

References

Details

(Whiteboard: [fixed-in-birch], QARegressExclude)

Attachments

(3 files, 1 obsolete file)

We should respond to remote device with ERROR when we received unsupported commands. On branch v1.0.1, it works for most cases, except AT+CHLD. Only CHLD=1 & CHLD=2 are supported, but we still reply OK for CHLD=n (n != 1 or 2)

Please see bug 848414 comment 13 for more information.
Assignee: nobody → gyeh
blocking-b2g: --- → tef?
Depends on: 848414
Nominated as tef? since Bug 848414 is a tef+.

Also, this bug is also related to Bluetooth Hfp certification. Should be land with the following bugs in order.

01. bug827204
02. bug827255
03. bug823346
04. bug827230
05. bug828798
06. bug827212, patch 1
06. bug827212, patch 2
07. bug827266
08. bug828175
09. bug846647
10. bug825861
11. bug825851
12. bug835740
blocking-b2g: tef? → tef+
Comment on attachment 739505 [details] [diff] [review]
Patch 1(v1): Send "ERROR" for unsupported value of AT+CHLD command

Review of attachment 739505 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good.
Attachment #739505 - Flags: review?(echou) → review+
Attachment #739505 - Attachment is obsolete: true
Whiteboard: [fixed-in-birch]
https://hg.mozilla.org/mozilla-central/rev/fe4689ed2edf
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Unable to test here. Need Bluetooth log sniff tool to verify fix this bug. Need to be able to read the logs as each command is sent from the bluetooth headset to the phone under test.

Marking as QARegressExclude.
Whiteboard: [fixed-in-birch] → [fixed-in-birch], QARegressExclude
You need to log in before you can comment on or make changes to this bug.