Closed Bug 1162224 Opened 5 years ago Closed 5 years ago

[STK] Browser Termination envelope not received


(Firefox OS Graveyard :: Gaia::System::SIM Tool Kit, defect)

Gonk (Firefox OS)
Not set


(blocking-b2g:2.2+, b2g-v2.2 fixed, b2g-master fixed)

2.2 S12 (15may)
blocking-b2g 2.2+
Tracking Status
b2g-v2.2 --- fixed
b2g-master --- fixed


(Reporter: anshulj, Assigned: selee)



(Whiteboard: [caf priority: p2][CR 833358])


(1 file)



- Card sends SET UP EVENT LIST: Browser Termination proactive command.
- Start the browser app
- Terminate the browser app by long pressing home button and closing the app

Expected: STK download event when the browser is terminated
Observed: No download event being sent
The issue seems to be that app.manifest.permissions.browser check at [1] is incorrect and therefore handleBrowserTerminationEvent function never gets called. Browser manifest url at [2] doesn't seem to have any entry for permissions.browser.


Just as an experiment I changed the Gaia code to look at | === 'Browser'| and then I received the envelope command as expected.
Whiteboard: [CR 833358]
Whiteboard: [CR 833358] → [caf priority: p2][CR 833358]
Flags: needinfo?(selee)
Triage: blocking
blocking-b2g: 2.2? → 2.2+
Comment on attachment 8602454 [details] [review]
[gaia] weilonge:seanlee/STK/master/Bug1162224 > mozilla-b2g:master

Hi Fernando,

Could you help to review the patch? Thank you.
Flags: needinfo?(selee)
Attachment #8602454 - Flags: review?(frsela)
Hi! Sean,

Since you are working on this case. Over to you. Thanks

Assignee: nobody → selee
Comment on attachment 8602454 [details] [review]
[gaia] weilonge:seanlee/STK/master/Bug1162224 > mozilla-b2g:master

r+ to the patch. Thanks for fixing

The browser app is changing since 2.0 (where permissions.browser exists [1]) and into the last releases it's merged inside system app [2].

I'm worried that using the name for checking it, is a weak solution because it can be changed in the future, but not sure about a better solution now :(

Attachment #8602454 - Flags: review?(frsela) → review+
Comment on attachment 8602454 [details] [review]
[gaia] weilonge:seanlee/STK/master/Bug1162224 > mozilla-b2g:master

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #):
[User impact] if declined:
Browser Termination envelope is not received correctly.

[Testing completed]:
Write unit-test for this and it passes.

[Risk to taking this patch] (and alternatives if risky):
Only change how to check Browser app terminated. Very minor.

[String changes made]:
Attachment #8602454 - Flags: approval-gaia-v2.2?
Attachment #8602454 - Flags: approval-gaia-v2.2? → approval-gaia-v2.2+
You need to log in before you can comment on or make changes to this bug.