1680827 - Duplicate geolocation last access timelabel in siteIdentity permission list

Status: RESOLVED FIXED

(Firefox :: Site Permissions, defect, P3)

Description

[Paul Zühlcke [:pbz]]

When clearing a permission via the site identity popup permission list the geolocation last access timelabel can duplicate.
This happens when clearing a webrtc permission with active sharing state. Clearing the webrtc permission causes the permission panel to be updated twice. This results in two last access timestamps for the geolocation entry, because fetching the last access timestamp is async. Both method invocations of _createGeoLocationLastAccessIndicator will append a timestamp item. The permission panel is cleared synchronously.

Steps:

1. Go to permission.site
2. Click on "geolocation" and confirm permission prompt
3. Click on "microphone" and confirm permission prompt
4. Open siteIdentity panel and remove microphone permission via the x button.

Expected result:

• the microphone permission has been removed
• the geolocation permission remains with one time entry

Actual result:

• the microphone permission has been removed
• the geolocation permission remains, but there are two time label entries

Comment 1

[Paul Zühlcke [:pbz]]

Attachment: Bug 1680827 - Don't create duplicate geolocation last access indicator. r=nhnt11

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f1e9e085cc68
Don't create duplicate geolocation last access indicator. r=nhnt11

Comment 3

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/f1e9e085cc68