Closed Bug 878006 Opened 7 years ago Closed 7 years ago

[Dialer] Should check call index of Bluetooth ATD command

Categories

(Firefox OS Graveyard :: Gaia::Dialer, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

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

RESOLVED FIXED
1.0.1 IOT3 (3jun)
blocking-b2g tef+
Tracking Status
firefox24 --- wontfix
b2g18 --- wontfix
b2g18-v1.0.0 --- wontfix
b2g18-v1.0.1 --- fixed

People

(Reporter: gyeh, Assigned: rudyl)

References

Details

Attachments

(1 file)

When there's a memory dialing request from Bluetooth headset, we will check call history in Dialer App, and then dial a new call according to the order of call history.

Two examples here.

1. There are 3 numbers in call history, and we receive a memory dialing request from Bluetooth headset, such as "ATD>2". Then, we should get the second call number from call history and make a new call.

2. There are 3 numbers in call history, and we receive a memory dialing request like "ATD>10". Then, ignore the message and no outgoing call is made.
Assignee: nobody → rlu
Nominate as tef+ since we need this (and bug 878005 as well, Gecko support) for BT certification. It's almost the last blocker. Thank you.
triage: tef+ to resolve BT cert blocker
blocking-b2g: tef? → tef+
Comment on attachment 756495 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/10131

Hi Etienne,

This simple patch adds support to dialer for handling call index of BT ATD command.

Please help review this with Bug 877070.
Thanks.
Attachment #756495 - Flags: review?(etienne)
batch update on tef+ milestones. partner to make a final on 6/3 Asia time. TEF+ needs to be resolved by 6/3 to be in the final build. thanks
Target Milestone: --- → 1.0.1 IOT3 (3jun)
Comment on attachment 756495 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/10131

r=me

Please squash before landing.

Thanks!
Attachment #756495 - Flags: review?(etienne) → review+
Comment on attachment 756495 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/10131

I'm very sorry, I got confused with bug 877070 (both were in the PR)

Please rebase this patch against the new one and ask for review again, thanks!
Attachment #756495 - Flags: review+
Comment on attachment 756495 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/10131

Hi Etienne,

The pull request has been updated.
Please note that this special action is for BT certification, and not used in general cases, that is why I did not add unit tests for it to get this done ASAP. Please let me know if you think it is still needed.

Thank you very much.

--
Really sorry to make you confused.
The original pull request includes both patches because they are highly related and the previous one was reviewed at that time.
Attachment #756495 - Flags: review?(etienne)
Comment on attachment 756495 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/10131

r=me with the nits on github addressed.

We'll definitely need unit tests for this but I'm willing to wait until the master version of the patch :)

We'll need a follow up bug for this (the master/v1-train) version too.

Thanks for all the bluetooth work!
Attachment #756495 - Flags: review?(etienne) → review+
Merged to Gaia master,
https://github.com/mozilla-b2g/gaia/commit/9ba891c880e588a8f3f8ca30bb932ecae68023a9

Etienne,
Thanks for the review.

--
Follow-up bug: Bug 878389 for v1-train/master.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
(In reply to Rudy Lu [:rudyl] from comment #10)
> Merged to Gaia master,

s/master/v1.0.1/

> https://github.com/mozilla-b2g/gaia/commit/
> 9ba891c880e588a8f3f8ca30bb932ecae68023a9
> 
> Etienne,
> Thanks for the review.
> 
> --
> Follow-up bug: Bug 878389 for v1-train/master.

Thanks for Eric's reminding.
:)
You need to log in before you can comment on or make changes to this bug.