Closed Bug 1096128 Opened 10 years ago Closed 10 years ago

[B2G][STK] support call number modified by STK call control

Categories

(Firefox OS Graveyard :: RIL, defect)

defect
Not set
normal

Tracking

(blocking-b2g:2.1+, firefox34 wontfix, firefox35 wontfix, firefox36 fixed, b2g-v2.0 wontfix, b2g-v2.0M fixed, b2g-v2.1 fixed, b2g-v2.2 fixed)

RESOLVED FIXED
2.1 S9 (21Nov)
blocking-b2g 2.1+
Tracking Status
firefox34 --- wontfix
firefox35 --- wontfix
firefox36 --- fixed
b2g-v2.0 --- wontfix
b2g-v2.0M --- fixed
b2g-v2.1 --- fixed
b2g-v2.2 --- fixed

People

(Reporter: jocheng, Assigned: hsinyi)

References

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #986308 +++ Created an attachment (id=671243) soul 3.5 fail log +++ This bug was initially created as a clone of Bug #409505 +++ DEFECT DESCRIPTION: [STK]call established not control by call control with modify command REPRODUCING PROCEDURES: 1.load one simulated SIM card and SATK application which can send "call control" command; 2.power on phone and run the case SATK->CALL control->STN08005->CC Alpha ID present; 3.dialling number not modify by control command--->KO from the network emulator, the call has already been modified ,but ME still keep display the old dial number. EXPECTED BEHAVIOUR: call control command can rightly execute ,ME will display the modified dial number. ASSOCIATE SPECIFICATION: TEST PLAN REFERENCE: TS spec 11.14 TOOLS AND PLATFORMS USED: V10I+AA. USER IMPACT: Middle. REPRODUCING RATE: 2/2. For FT PR, Please list reference mobile's behavior: ++++++++++ end of initial bug #409505 description ++++++++++
Hi Bhavana, This is certification blocker for TCL per https://bugzilla.mozilla.org/show_bug.cgi?id=986308#c11. Hsin-Yi has provided patch for 2.0M and m/c on bug 986308. Could you assess whether we should also fix this in 2.1? Thank you!
Flags: needinfo?(bbajaj)
Summary: [B2G][STK][Woodduck] support call number modified by STK call control → [B2G][STK] support call number modified by STK call control
(In reply to Josh Cheng [:josh] from comment #1) > Hi Bhavana, > This is certification blocker for TCL per > https://bugzilla.mozilla.org/show_bug.cgi?id=986308#c11. > Hsin-Yi has provided patch for 2.0M and m/c on bug 986308. > Could you assess whether we should also fix this in 2.1? Thank you! Given this is a partner cert blocker its fine to land. But given the patches in 986308 fix the issue and have changes to ril_worker.js, Anshul can you help comment is these changes are fine to be uplifted on CAF side. Also NI :hsinyi to help assess the risk of uplift for patches in 986308
Flags: needinfo?(htsai)
Flags: needinfo?(bbajaj)
Flags: needinfo?(anshulj)
Bhavna, yes it is fine to be uplifted to 2.1.
Flags: needinfo?(anshulj)
Thanks Anshul. Hsinyi you have a chance to comment on the risk here and think its manageable please seek approval for uplifts for b2g34 branch.
(In reply to bhavana bajaj [:bajaj] from comment #5) > Thanks Anshul. Hsinyi you have a chance to comment on the risk here and > think its manageable please seek approval for uplifts for b2g34 branch. The risk is low given the changes are straight-forward. I'll provide a b2g34 patch later, thanks.
NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings. [Approval Request Comment] Bug caused by (feature/regressing bug #): bug 986308, a certification blocker User impact if declined: certification fails and user cannot get the right call number Testing completed: yes, and also landed on m-c, 2.0m already Risk to taking this patch (and alternatives if risky): low String or UUID changes made by this patch: no
Flags: needinfo?(htsai)
Attachment #8525028 - Flags: approval-mozilla-b2g34?
Attachment #8525028 - Flags: approval-mozilla-b2g34? → approval-mozilla-b2g34+
blocking-b2g: --- → 2.1+
checkin-needed for *v2.1* only (landed on m-c already in bug 986308) https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=61f4d68dc781 try looks good
Keywords: checkin-needed
NI :kwierso for help on checkin-needed given ryan is out for a bit.
Flags: needinfo?(kwierso)
Pushed to b2g34_v2_1 in https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/d7b12401107c This doesn't need to get into Firefox 34 (mozilla-beta), correct?
Flags: needinfo?(kwierso)
Keywords: checkin-needed
(In reply to Wes Kocher (:KWierso) from comment #11) > Pushed to b2g34_v2_1 in > https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/d7b12401107c > > This doesn't need to get into Firefox 34 (mozilla-beta), correct? correct, let me wontfix it.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee: nobody → htsai
Target Milestone: --- → 2.1 S9 (21Nov)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: