[fenix] removed auto play permissions while determining isVisible for WebsitePermissionsState

pull/600/head
apoorv9990 4 years ago committed by Emily Kager
parent d771f5ae43
commit bc7cd19116

@ -145,8 +145,7 @@ class QuickSettingsFragmentStore(
val autoplayInaudiblePermission = val autoplayInaudiblePermission =
PhoneFeature.AUTOPLAY_INAUDIBLE.toWebsitePermission(context, permissions, settings) PhoneFeature.AUTOPLAY_INAUDIBLE.toWebsitePermission(context, permissions, settings)
val shouldBeVisible = cameraPermission.isVisible || microphonePermission.isVisible || val shouldBeVisible = cameraPermission.isVisible || microphonePermission.isVisible ||
notificationPermission.isVisible || locationPermission.isVisible || notificationPermission.isVisible || locationPermission.isVisible
autoplayAudiblePermission.isVisible || autoplayInaudiblePermission.isVisible
return WebsitePermissionsState( return WebsitePermissionsState(
shouldBeVisible, cameraPermission, microphonePermission, shouldBeVisible, cameraPermission, microphonePermission,

@ -55,12 +55,7 @@ class WebsitePermissionsView(
* @param state [WebsitePermissionsState] to be rendered. * @param state [WebsitePermissionsState] to be rendered.
*/ */
fun update(state: WebsitePermissionsState) { fun update(state: WebsitePermissionsState) {
val isAnyPermissionVisible = state.camera.isVisible || state.location.isVisible || if (state.isVisible) {
state.microphone.isVisible || state.notification.isVisible
// Can not use state.isVisible because we are not handling the audio permissions here right
// now. If we add more permissions below we should update isAnyPermissionVisible too
if (isAnyPermissionVisible) {
interactor.onPermissionsShown() interactor.onPermissionsShown()
} }

Loading…
Cancel
Save