838347 - Installing a privileged packaged app requesting all PROMPT_ACTION permissions - only geolocation is listed in the settings app if it hasn't been launched

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::Settings, defect)

Description

[Jason Smith [:jsmith]]

Build: B2G 18 2/5/2013
Device: Unagi

Steps:

1. Go to https://marketplace.firefox.com/app/test-webapi-permissions-3
2. Install it
3. Go the settings app --> app permissions --> <app name>

Expected:

All PROMPT_ACTION permissions should be listed that were specified in the app manifest.

Actual:

Only geolocation is listed. This is obviously incorrect - we should be listing each PROMPT_ACTION permission provided.

Comment 1

[Jason Smith [:jsmith]]

Gregor mentions that Reuben has been touching this area of the codebase recently. Reuben - Do you have any ideas what might have broke this?

Comment 2

[Jason Smith [:jsmith]]

Also - do we know if v1.0.0 is affected? I know for a fact v1-train is affected given that this is the build I'm testing on.

If it's v1.0.0 affected, this should be tef+. If it's v1-train affected only, then this should be shira+.

Comment 3

[Gregor Wagner [:gwagner]]

Reubens patch is not in b2g18. He was working on 793204.

Comment 4

[Jason Smith [:jsmith]]

(In reply to Gregor Wagner [:gwagner] from comment #3)
> Reubens patch is not in b2g18. He was working on 793204.

Oh. In that case, then it can't be Reuben's patch - this is reproducing on b2g18. Reuben's patch is only on central.

Comment 5

[Jason Smith [:jsmith]]

Confirmed this is busted on v1.0.0 as well on 2/5 build.

Comment 6

[Lukas Blakk [:lsblakk] use ?needinfo]

Gregor can you take this or do you know someone else who can?  We'll need to repair this regression and are blocking tef on it, so it has to have an active assignee. Thank you.

Comment 7

[Gregor Wagner [:gwagner]]

It works on the desktop build but not on the device :(

Comment 8

[Gregor Wagner [:gwagner]]

(In reply to Gregor Wagner [:gwagner] from comment #7)
> It works on the desktop build but not on the device :(

Hm I just tested on the unagi and it also works.
jsmith can you try again?
I am using todays gaia and b2g18

The 2nd time I wanted to install it, one process crashed. Maybe that's related here.

Comment 9

[Gregor Wagner [:gwagner]]

I can't test this right now because bug 839688 is causing random crashes during install. waiting for the backout.

Comment 10

[Jason Smith [:jsmith]]

Dug into this. Here's what going on here:

When I install a privileged packaged app and immediately check app perms UI, geolocation is indeed only shown. However! If I launch the app, then check the app perms UI, then the right perms are listed. So I'm now wondering if this is a UI bug.

I also now don't think this blocks either - I strongly doubt a user will hit this scenario and there's an easy workaround.

Comment 11

[Gregor Wagner [:gwagner]]

(In reply to Jason Smith [:jsmith] from comment #10)
> Dug into this. Here's what going on here:
> 
> When I install a privileged packaged app and immediately check app perms UI,
> geolocation is indeed only shown. However! If I launch the app, then check
> the app perms UI, then the right perms are listed. So I'm now wondering if
> this is a UI bug.
> 
> I also now don't think this blocks either - I strongly doubt a user will hit
> this scenario and there's an easy workaround.

Thanks for checking! That doesn't sound like a blocking bug.

Comment 12

[Alex Keybl [:akeybl]]

(In reply to Jason Smith [:jsmith] from comment #10)
> I also now don't think this blocks either - I strongly doubt a user will hit
> this scenario and there's an easy workaround.

Agreed, not a blocker.

Comment 13

[Lukas Blakk [:lsblakk] use ?needinfo]

Batch edit: Bugs marked status-b2g18: affected after 2/13 branching of v1.0.1 are now also status-b2g18-v1.0.1: affected

Comment 14

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

http://bit.ly/13llomx

apps permission crashes after install on 2012_12_27_23 build.
2013-01-07 web api install still broken.

-> Works on 2013-01-10-23-02-02: all APIs show
"releases/mozilla-b2g18" revision="82acdace6740
"integration/gaia-nightly" revision="c9a778d8d9b5"

"releases/gecko.git" revision="a84a3754648cce1a240caa31860cd99bcefac573"
"releases/gaia.git" revision="9d4983b18d5ff79b5900a8075d38eea352635728"

-> Actual result shows on 2013-01-11-23-02-01  : only Geo location shows
"releases/mozilla-b2g18" revision="8f2ef4998b60"
"integration/gaia-nightly" revision="45a3196a5517"

"releases/gecko.git" revision="3ce75653635cdeae69253320e45157171544c596"
"releases/gaia.git" revision="df38c1bb813029f3ccfa4a997fb1529b3ff1a1ff"

Comment 15

[BMO Automation]

Firefox OS is not being worked on