Closed Bug 1528303 Opened 6 years ago Closed 6 years ago

After denying a microphone Android permission I'm getting a call to onMediaPermissionRequest

Categories

(GeckoView :: Media, defect, P1)

Unspecified
Android
defect

Tracking

(firefox-esr60 wontfix, firefox65 wontfix, firefox66 wontfix, firefox67 fixed)

RESOLVED FIXED
mozilla67
Tracking Status
firefox-esr60 --- wontfix
firefox65 --- wontfix
firefox66 --- wontfix
firefox67 --- fixed

People

(Reporter: amejia, Assigned: fluffyemily)

References

Details

(Whiteboard: [geckoview:fenix:m3])

Attachments

(1 file)

After denying a microphone Android permission (onAndroidPermissionsRequest) , I'm getting a call to onMediaPermissionRequest and I can even grant the permission.

This can be replicated on the GeckoView Example app.

Steps to reproduce

  1. Go to https://permission.site/
  2. Tap on the Microphone button
  3. Tap on the "deny" button of the Android permission dialog.
  4. Tap on the ok button of the site permission dialog.

Expected behavior

onMediaPermissionRequest must not be called after denying the Android permission.

Actual behavior

onMediaPermissionRequest is getting call after denying the Android permission.

Assignee: nobody → etoop
Priority: -- → P1
Whiteboard: [geckoview:fenix:m3]

For Camera, we save the android permission choice and then suppress the subsequent Media permission, but we don't do this for microphone access. We should treat these permissions the same. As we want to move to a situation where GV doesn't maintain any permissions state I shall move Camera to behave the same as the Microphone. This means that it is up to the consumer to remember any state and behave accordingly with regards to issuing a prompt for Media permissions.

I shall update GVE to reflect the actions that consumers would have to perform in order to produce the behaviour expected by permissions.site. It is recommended that Android Components handles this case.

Update onMediaPermissionRequest documentation to better reflect usage

Pushed by etoop@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b4eb5531f8e7 Update GVE to demonstrate how to handle media permissions correctly r=geckoview-reviewers,snorp
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67

Moving some media bugs to the new GeckoView::Media component.

Component: General → Media
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: