Closed Bug 1235110 Opened 9 years ago Closed 9 years ago

[CBS] Forbid the DATA DOWNLOAD message to be delivered to the App layer.

Categories

(Firefox OS Graveyard :: RIL, defect)

Product:
Firefox OS Graveyard
Component:
RIL
Platform:
ARM
Gonk (Firefox OS)
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox46 fixed, b2g-v2.5 affected, b2g-master affected)

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
firefox46 --- fixed
b2g-v2.5 --- affected
b2g-master --- affected

People

(Reporter: bevis, Assigned: bevis)

References

Details

(Keywords: foxfood)

Attachments

(2 files)

(v1) Part 1: Forbid the DATA DOWNLOAD message to be delivered to the App layer.
(deleted), patch
edgar
: review+
Details | Diff | Splinter Review
(v1) Part 2: Add Test Coverage.
(deleted), patch
edgar
: review+
Details | Diff | Splinter Review
This is a follow-up of bug 987233 comment 8. The CB messages for SIM DOWNLOAD are expected to be handled directly by the modem to update the data in SIM via STK SIM DOWNLOAD protocol. However, the modem still notify these messages to the application layer. File this bug to filter out this kind of CB messages.
Blocks: 987233
No longer blocks: 987233
BTW, according the 3GPP requirement, the the channels of these SIM DOWNLOAD message are not allowed to be disabled from the UI settings, that's why the device still received these messages when the alert is disabled.
[Blocking Requested - why for this release]: This is annoying to the foxfooding user according to bug 987233 comment 8.
blocking-b2g: 2.6? → 2.5?
Summary: [CBS] Forbid the SIM DOWNLOAD message to be delivered to the App layer. → [CBS] Forbid the DATA DOWNLOAD message to be delivered to the App layer.
Hi Edgar, May I have your review for this change? Thanks!
Attachment #8702863 - Flags: review?(echen)
Attached patch (v1) Part 2: Add Test Coverage. (deleted) — — Splinter Review
Attachment #8702864 - Flags: review?(echen)
Blocks: b2g-cbs
Attachment #8702863 - Flags: review?(echen) → review+
Comment on attachment 8702864 [details] [diff] [review] (v1) Part 2: Add Test Coverage. Review of attachment 8702864 [details] [diff] [review]: ----------------------------------------------------------------- Thank you.
Attachment #8702864 - Flags: review?(echen) → review+
Set block to bug 1235697 because the change is inside the same function.
Blocks: 1235697
Per the drivers meeting this week, we won't uplift to v2.5 if it's a non-TV blocker. Cancelling the flag.
blocking-b2g: 2.5? → ---
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: