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
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.
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) — Splinter Review
Attached patch 917922.patchSplinter 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: