Closed Bug 977920 Opened 7 years ago Closed 7 years ago
Service should not check for ns IDOMMoz Gsm Icc Info when retrieving icc Id
Reference: http://dxr.mozilla.org/mozilla-central/source/dom/mobilemessage/src/gonk/MmsService.js#318 Since iccId applies to both GSM and CDMA, seems like this check should be removed: !(iccInfo instanceof Ci.nsIDOMMozGsmIccInfo)
What's the user impact of this bug? Why is this needed for 1.4 FC?
(In reply to Jason Smith [:jsmith] from comment #1) > What's the user impact of this bug? Why is this needed for 1.4 FC? Please note that bugzilla 975356 will now broadcast iccId for both incoming MMS and SMS. Without the fix suggested in comment 0, we would not know which SIM the message is received on, which ultimately breaks bugzilla 947180.
blocking-b2g: 1.4? → 1.4+
Component: General → DOM: Device Interfaces
Product: Firefox OS → Core
Version: unspecified → Trunk
This is for Bevis
Component: DOM: Device Interfaces → RIL
Product: Core → Firefox OS
Version: Trunk → unspecified
Assignee: nobody → btseng
The same symptom in RadioInterfaceLayer.js: http://dxr.mozilla.org/mozilla-central/source/dom/system/gonk/RadioInterfaceLayer.js#2296
Tested with both USIM/RUIM. The IccId could be retrieved correctly without check the prototype of IccInfo.
Attachment #8385872 - Flags: review?(vyang)
Attachment #8385872 - Flags: review?(vyang) → review+
Target Milestone: --- → 1.4 S3 (14mar)
try sever result is green. https://tbpl.mozilla.org/?tree=Try&rev=54df34276b47
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
No STR is present to create test case to address bug.
QA Whiteboard: [QAnalyst-Triage?]
Not enough information to create a test case.
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
You need to log in before you can comment on or make changes to this bug.