Closed Bug 1132007 Opened 9 years ago Closed 9 years ago

Initialise _iccChannelCalback in RilContentHelper.js


(Firefox OS Graveyard :: RIL, defect)

Not set


(blocking-b2g:2.2+, firefox36 wontfix, firefox37 wontfix, firefox38 fixed, b2g-v2.2 fixed, b2g-master fixed)

2.2 S6 (20feb)
blocking-b2g 2.2+
Tracking Status
firefox36 --- wontfix
firefox37 --- wontfix
firefox38 --- fixed
b2g-v2.2 --- fixed
b2g-master --- fixed


(Reporter: tauzen, Assigned: tauzen)




(1 file, 1 obsolete file)

While running SecureElement code (Bug 879861) on master I've got "this._iccChannelCallback is undefined" error. _iccChannelCallback was introduced in Bug 1081789, and the original version of the patch actually initialised it. The initialisation line was probably lost while rebasing to master. It needs to be added back for SE API to work properly.
Blocks: 879861
blocking-b2g: --- → 2.2?
Depends on: 1081789
Hi Edgar, since you reviewed the patches in Bug 1081789 could you take a look a this? From my perspective this looks like a regression caused by rebase.
Attachment #8562758 - Flags: review?(echen)
Assignee: nobody → kmioduszewski
triage: blocker as it's blocking bug 879861 (SE)
blocking-b2g: 2.2? → 2.2+
Comment on attachment 8562758 [details] [diff] [review]

Review of attachment 8562758 [details] [diff] [review]:

r=me with comment addressed. Thank you, kmioduszewski!

Please also help to add `_iccChannelCallback: null` right before `_addIccChannelCallback: function(requestId, channelCb) {` [1], just like what |_iccListeners| does [2].

Attachment #8562758 - Flags: review?(echen) → review+
Thanks Edgar! I've updated the patch as you requested. Once I'll get a green try build I'll request checkin.
Attachment #8562758 - Attachment is obsolete: true
Attachment #8563340 - Flags: review+

I think there's a global problem with OSX builds, apart from that everything seems to be ok.
Keywords: checkin-needed
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → 2.2 S6 (20feb)
Comment on attachment 8563340 [details] [diff] [review]
(v1.1) 0001-Bug-1132007-Initialise-_iccChannelCalback-in-RilCont.patch

NOTE: Please see to better understand the B2G approval process and landings.

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
Bug 1132007, see more details in comment 0
User impact if declined:
Broken Secure Element API feature - Bug 879861  
Testing completed: on device testing complete
Risk to taking this patch (and alternatives if risky): none
String or UUID changes made by this patch: none
Attachment #8563340 - Flags: approval-mozilla-b2g37?
Attachment #8563340 - Flags: approval-mozilla-b2g37? → approval-mozilla-b2g37+
You need to log in before you can comment on or make changes to this bug.