907075 - [Permission] Dismiss continuous same permission request but dispatch mozContentEvent as well if remember is checked

Status: VERIFIED FIXED

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

Description

[(inactive after 6/18) Alive Kuo [:alive]]

Use case:
(1) User app requests permission A.
(2) Permission Screen shows.
(3) Before user clicks yes or no, user app requests permission A again for other purpose.
(4) User clicks yes and remember me.

Expected:
Don't show the second dialog but sends the ID to gecko as well.

Note:
This affects v1.2 audio recording feature.

Comment 1

[(inactive after 6/18) Alive Kuo [:alive]]

Hi, before doing this I still needs some gecko change to tell me "who is asking permission?" because I cannot cancel another app's request and currently I don't know which app is asking permission.

I think provide manifestURL and pageURL in the permission-request mozChromeEvent is sufficient.

Comment 2

[(inactive after 6/18) Alive Kuo [:alive]]

Hmm, due to current permission screen design, this one may not be a trivial change.
I'd like to mentor some new comer to work on refactoring of permission screen. Any way I could also still do this on my own.

Comment 3

[[e-mail]]

per discussion, bug 853356 is the related use scenario

Comment 4

[[e-mail]]

another related bug 898949

Comment 5

[[e-mail]]

@alive, does bug 786203 provide sufficient origin info?

https://hg.mozilla.org/mozilla-central/rev/0b02b848613e

Comment 6

[[e-mail]]

will work on bug 913896 first which is a critical path to support gUM

Comment 7

[[e-mail]]

Attachment: pull request redirect to github

@alive, is this as you expect?

I'd like to write related test case and test on real device if I'm in the right path.

Comment 8

[[e-mail]]

Comment on attachment 801478 [details]
pull request redirect to github

Thanks for feedback (in github), I've add testcases for review

Comment 9

[(inactive after 6/18) Alive Kuo [:alive]]

Comment on attachment 801478 [details]
pull request redirect to github

r+. Have you tested on the real device sending continuous request?

Comment 10

[[e-mail]]

not test in real device yet.

@alfredo, is there any exist demo case for v1.2 audio recording feature?

Comment 11

[[e-mail]]

merged in gaia-master https://github.com/mozilla-b2g/gaia/commit/dcc3707403f43da01f6072f1b5e3a144ee4aaf61

would like to solve followup issues(if any) in bug 913896

Comment 12

[Jason Smith [:jsmith]]

Verified through initial gUM test pass using http://mozilla.github.io/qa-testcase-data/webapi/webrtc/multigumaudiocontent.html.

Comment 13

[Alfredo Yang (:alfredo)]

Just seen this.
I don't think it needs to use audio recorder, just multiple of gUM calls should be sufficient.