Closed Bug 917922 Opened 11 years ago Closed 11 years ago

[B2G] [Buri] [Dialer] temporary CLIR mode not work

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(blocking-b2g:koi+, firefox25 wontfix, firefox26 fixed, firefox27 fixed, b2g-v1.2 fixed)

RESOLVED FIXED
blocking-b2g koi+
Tracking Status
firefox25 --- wontfix
firefox26 --- fixed
firefox27 --- fixed
b2g-v1.2 --- fixed

People

(Reporter: ckreinbring, Assigned: hsinyi)

References

Details

(Keywords: regression, Whiteboard: burirun1 [comms-triage])

Attachments

(3 files, 1 obsolete file)

Attached file Log of CLIR attempt (deleted) —
Description: A user that attempts to use a CLIR code before dialing a number will find that the Call button is unresponsive. Repro Steps: 1) Update Buri to Build ID: 20130916040205 2) Launch the Phone app. 3) Enter a CLIR code (eg *#31#). 4) Tap the Call button and observe the device's reaction. Actual: There is no reaction. Expected: The code is processed and the user is shown the proper result of the code. Environmental Variables Occurs on Buri 1.2 Mozilla RIL Build ID: 20130916040205 Gecko: http://hg.mozilla.org/mozilla-central/rev/c4bcef90cef9 Gaia: a0079597d510ce8ea0b9cbb02c506030510b9eeb Platform Version: 26.0a1 Notes: Repro frequency: 100% Test Suite Name: Dialer UCID: owd-28087 Link to failed test case: https://moztrap.mozilla.org/manage/cases/?filter-id=9571 See attached logcat logs Once a call has been made to another number, CLIR codes can be sent.
No longer blocks: b2g-central-dogfood
blocking-b2g: --- → koi?
QA Contact: sparsons
This issue started to occur on the 9/09 Buri 1.2 Build ID: 20130909114657 Environmental Variables Build ID: 20130909114657 Gecko: http://hg.mozilla.org/mozilla-central/rev/218d4334d29e Gaia: aa4180e9286d385fa6b62d236f30fb24cd8b93e9 Platform Version: 26.0a1 RIL Version: 01.02.00.019.043 Last working build 9/06 Buri 1.2 Build ID: 20130906040204 Environmental Variables Build ID: 20130906040204 Gecko: http://hg.mozilla.org/mozilla-central/rev/ab5f29823236 Gaia: 94e5f269874b02ac0ea796b64ab995fce9efa4b3 Platform Version: 26.0a1 RIL Version: 01.02.00.019.043
Works for me with latest m-i (148026) and MOZ RIL
(In reply to Fernando Jiménez Moreno [:ferjm] (needinfo, please) from comment #2) > Works for me with latest m-i (148026) and MOZ RIL Works for me, too. gecko (mozillaorg/master): e55134 (Sep. 23)
Blocks: 921979
Found similar issue on Buri that call can be dialed out yet number is not hidden. Gaia: 1e9470b9b6df630eddf1c4c8b25b3170ee786b0e Gecko: http://hg.mozilla.org/releases/mozilla-aurora/rev/48faa2668dd8 BuildID 20130929004004 Version 26.0a2 Log will be attached, please let me know if another bug is needed or we can track CLIR issue here. Steps: 1. Use FET carrier account in Taiwan 2. Dial #31#09xxxxxxxx (#31# can hide MO side number) 3. MT side can still see DUT's number.
Attached file Buri, time stamp: 15:40, 10/1. (deleted) —
hsinyi or fernando, do you mind looking at the logs to help the triage decide if this is a blocker or not? thanks
Whiteboard: burirun1 → burirun1 [comms-triage]
Quick debug observation: We somehow misinterpreted the meaning *31 and #31 in gecko code. We have been using *31 to hide the number rather than #31 that is wrong.
Hi HsinYi, I wonder if this is because CDMA use *31# to hide number under APTG network. Just to let you know that *31# still needed for CDMA. But if it has nothing to do with design for CDMA, then it's fine. Thanks.
(In reply to Enpei from comment #8) > Hi HsinYi, > > I wonder if this is because CDMA use *31# to hide number under APTG network. > Just to let you know that *31# still needed for CDMA. But if it has nothing > to do with design for CDMA, then it's fine. > > Thanks. Enpei, Thanks for the information. :) We have divergent ways for CDMA and GSM in this case. They should not interact.
Summary: [B2G] [Buri] [Dialer] CLIR codes cannot be entered before a number has been called → [B2G] [Buri] [Dialer] temporary CLIR mode not work
Assignee: nobody → htsai
Attached patch 917922.patch (obsolete) (deleted) — Splinter Review
Attached patch 917922.patch (deleted) — Splinter Review
Removing _getCLIRMode() since it repeats most checks in _isTemporaryModeCLIR().
Attachment #812979 - Attachment is obsolete: true
Comment on attachment 812981 [details] [diff] [review] 917922.patch Hi Fernanrdo, Would you mind taking a look at this? Thank you.
Attachment #812981 - Flags: review?(ferjmoreno)
blocking-b2g: koi? → koi+
Comment on attachment 812981 [details] [diff] [review] 917922.patch Review of attachment 812981 [details] [diff] [review]: ----------------------------------------------------------------- Thanks Hsin-Yi! Please, make sure that test_ril_worker_clir.js test passes before landing.
Attachment #812981 - Flags: review?(ferjmoreno) → review+
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: