914028 - Display video (camera) device list in permission acquisition prompt

Status: RESOLVED FIXED

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

Description

[[e-mail]]

To support Display camera permission acquisition prompt,

Permission Manager need to show the available device list get from gecko, with single select option box near it.

Comment 1

[[e-mail]]

Attachment: UX mockup

add UX mockup for reference, will take l10n into concern

Comment 2

[[e-mail]]

Attachment: UX mockup with 'remember' option

Comment 3

[[e-mail]]

Attachment: pull request redirect to github

WIP, wait gecko ready to test

Comment 4

[[e-mail]]

Attachment: WIP screenshot

Comment 5

[Ivan Tsay (:ITsay)]

Plus it because it blocked the gUM permission on video/camera.

Comment 6

[Jason Smith [:jsmith]]

Per discussion offline - gUM camera support is a targeted feature, not committed, which means we don't need to block on this. We should try to get this in for 1.3 still, however.

Comment 7

[[e-mail]]

I'd suggest add one edge case for usability: 

hide the option list if there's only 1 option, just set that option as default choice (ex: unagi or inari only have back camera)

Patryk, how do you think?

Comment 8

[Patryk Adamczyk [:patryk] UX]

Agreed. If there is only one camera (usually the rear), you shouldn't need to see the list, its redundant. The user just needs to see the question: "Would like like to share you camera?"

Comment 9

[[e-mail]]

Comment on attachment 8333744 [details] [review]
pull request redirect to github

Test with SC's test-version gecko and everything seems fine now.

1. Form the list if there's more than 1 options
2. single selection
3. wider selection area that user can tape anywhere to select the line of option
4. fix 'remember' issue by initialize variables in the very beginning

Comment 10

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

Comment on attachment 8333744 [details] [review]
pull request redirect to github

r+ if function is split.

Comment 11

[[e-mail]]

I'll send a separately patch for some base fix in 1.3, and land this list support on 1.4.

Comment 12

[[e-mail]]

After some function split, it seems works well in SC's test-version gecko 

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