Closed Bug 952244 Opened 6 years ago Closed 6 years ago

Remember my choice is showing up again on gUM audio permission prompts

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(blocking-b2g:1.3+, b2g-v1.3 verified)

RESOLVED FIXED
blocking-b2g 1.3+
Tracking Status
b2g-v1.3 --- verified

People

(Reporter: jsmith, Assigned: gasolin)

References

Details

(Keywords: regression, Whiteboard: [FT:System-Platform])

Attachments

(1 file)

The remember my choice option has returned on gUM audio permission prompts. bug 917359 intentionally disabled this for security reasons. It needs to be disabled again on 1.3 in all cases. In 1.4 however, it should only be disabled in the browser, as the required security dependency in bug 945111 has been fixed to allow remember my choice to be enabled in apps.
For doing the regression window here:

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

Expected

No remember my choice option in perm prompt.

Actual

Remember my choice option is present in perm prompt.
blocking-b2g: --- → 1.3?
Whiteboard: [FT:System-Platform]
QA Contact: jzimbrick
Regression Window:

Last Working Environmental Variables:
Device: Buri v1.3 Mozilla RIL
BuildID: 20131113040200
Gaia: 7243b75041c93bc7deb240131748d0b164f3f0b0
Gecko: 7b014f0f3b03
Version: 28.0a1
Base Image: V1.2_20131115

First Broken Environmental Variables:
Device: Buri v1.3 Mozilla RIL
BuildID: 20131114143149
Gaia: 4ea5adac92d6a9ab035e708e941d50be0d670da2
Gecko: 7b014f0f3b03
Version: 28.0a1
Base Image: V1.2_20131115
look on it
Assignee: nobody → gasolin
bug 936997 seems suspicious for causing this.
I believe the cause is now we are working on `backward compatible mode` in permission manager since bug 853356 is backed out..
The evidence is you'll see 'XXX version < v1.2 does not support...' in console log.

We can wait for bug 853356 landed or add more backward compatible code for this case.

@SC, could I wait for bug 853356 landed?
Depends on: 853356
Flags: needinfo?(schien)
(In reply to Fred Lin [:gasolin] from comment #5)
> I believe the cause is now we are working on `backward compatible mode` in
> permission manager since bug 853356 is backed out..
> The evidence is you'll see 'XXX version < v1.2 does not support...' in
> console log.
> 
> We can wait for bug 853356 landed or add more backward compatible code for
> this case.
> 
> @SC, could I wait for bug 853356 landed?

Can't. That bug has no plans to land in 1.3.
looks like an easy fix, so add PR for review
Attachment #8350434 - Flags: review?(alive)
Flags: needinfo?(schien)
Removing dependency on gUM camera perm prompt as this can't be dependent on that feature, as that will not land for 1.3.
No longer depends on: 853356
Comment on attachment 8350434 [details] [review]
pull request redirect to github

Since this happens again I'd like to see any test written for this.
Attachment #8350434 - Flags: review?(alive)
Comment on attachment 8350434 [details] [review]
pull request redirect to github

add testcases, please kindly review it again. Thanks
Attachment #8350434 - Flags: review?(alive)
Triage: regression of audio prompt.
blocking-b2g: 1.3? → 1.3+
Attachment #8350434 - Flags: review?(alive) → review+
merged to gaia-master https://github.com/mozilla-b2g/gaia/commit/1584816093182199576467b63d8c7e19c035a3a8
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Uplifted 1584816093182199576467b63d8c7e19c035a3a8 to:
v1.3: af2b5ce2e3887c61d44433d47d17219b9082c705
lgtm on a 1.3 build from 1/8/2014
You need to log in before you can comment on or make changes to this bug.