917359 - Remember my choice is exposed for the getUserMedia dialog, but shouldn't be

Status: RESOLVED FIXED

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

Description

[Jason Smith [:jsmith]]

Build: 9/17/2013 Master
Device: Inari

STR

1. Go to http://mozilla.github.io/webrtc-landing/gum_test.html
2. Select audio

Expected

The permission prompt should appear with no remember my choice option.

Actual

The permission prompt appears with a remember my choice option. It appears to be functional as well. This is unsafe to expose to the web in general - remember my choice was not supposed to be exposed right now generally for gUM due to privacy concerns.

Comment 1

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

I agree this might be a concern, may we get UX confirm on this? There's remember my choice in the design document.

BTW shall we distinguish web and app in this case?

Comment 2

[Jason Smith [:jsmith]]

(In reply to Alive Kuo [:alive] from comment #1)
> I agree this might be a concern, may we get UX confirm on this? There's
> remember my choice in the design document.

See https://bugzilla.mozilla.org/show_bug.cgi?id=898986#c3. Remember my choice cannot be implemented generally for all permission cases because a user would be required to have an easy path to forgot my choice for privacy reasons. jesup & pauljt could give more information on this if need be.

> 
> BTW shall we distinguish web and app in this case?

In bug 898986, we concluded enabling remember my choice for an app was on the table as an option potentially if the settings app perms UI grants control audio/video capture permissions. When I tested this implementation, it looks like the settings apps permissions implementation is not updated to include audio/video capture permissions, which means we can't enable remember my choice safely yet. Given that we're past feature complete, that likely means we're going to have to go with the decision to disable remember my choice across the board then to meeting the privacy requirements we have.

Comment 3

[[e-mail]]

Attachment: pull request redirect to github

WIP, consider done but need test once I have better wifi.
Will send review after test passed

Comment 4

[bhavana bajaj [:bajaj]]

HI Fred,

A quick ping to check on an estimate when this issue will be resolved.I see you have a WIP patch in comment #3. All set for review ?

Comment 5

[[e-mail]]

I think it can be done this week.

Comment 6

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

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

r+ with github comment

Comment 7

[[e-mail]]

merged to gaia-master https://github.com/mozilla-b2g/gaia/commit/d977e335e9bef01dae60713ffac12922fc32c35b

thanks!

Comment 8

[John Ford [:jhford] CET/CEST Berlin Time]

I was not able to uplift this bug to v1.2.  If this bug has dependencies which are not marked in this bug, please comment on this bug.  If this bug depends on patches that aren't approved for v1.2, we need to re-evaluate the approval.  Otherwise, if this is just a merge conflict, you might be able to resolve it with:

  git checkout v1.2
  git cherry-pick -x -m1 d977e335e9bef01dae60713ffac12922fc32c35b
  <RESOLVE MERGE CONFLICTS>
  git commit

Comment 9

[[e-mail]]

Thanks for reminding.

This bug depends on bug 915468, bug 922713, which should be uplift to 1.2. Add approval-1.2 request to those bugs

Comment 10

[John Ford [:jhford] CET/CEST Berlin Time]

Uplifted d977e335e9bef01dae60713ffac12922fc32c35b to:
v1.2: 15b4458cda572ca9a277f52fe7588381d166161f

Comment 11

[Jason Smith [:jsmith]]

Verified on a 1.2 11/1 build that there no longer is a remember my choice option on the gUM audio perm prompt.