659087 - "Always Ask" defaults for geolocation and indexedDB don't work

Status: VERIFIED FIXED

(Firefox :: Settings UI, defect)

Description

[:Margaret Leibovic]

The DefaultPermissions geo and indexedDB setters check for an ALLOW value to enable these permissions, but this is wrong because enabling these permissions really just means that they aren't blocked by default. Also, in the UI there is no "allow" menuitem for these permissions, so there's no way to un-block them.

Comment 1

[:Margaret Leibovic]

Attachment: patch

Comment 2

[:Gavin Sharp [email: [e-mail]]]

Comment on attachment 534522 [details] [diff] [review]
patch

Can you make the same change to the passwords case, as well? I think we also need a followup for properly using "always ask" vs. "allow" in that case, so this change should help (and be equivalent in the mean time).

Another minor thing I noticed that would be good to fix in this patch, even though it's unrelated:

> if (Services.prefs.getIntPref("network.cookie.lifetimePolicy") == this.COOKIE_DENY) {

should be:

if (Services.prefs.getIntPref("network.cookie.lifetimePolicy") == this.COOKIE_SESSION) {

Comment 3

[:Margaret Leibovic]

Attachment: patch for check-in

(In reply to comment #2)
> I think we also
> need a followup for properly using "always ask" vs. "allow" in that case

Filed bug 659319.

Comment 4

[:Margaret Leibovic]

Comment on attachment 534769 [details] [diff] [review]
patch for check-in

I didn't have the chance to land this on mozilla-central yet due to tree craziness, but I think it's a valuable fix that would be good to land on aurora sooner rather than later. Without this patch, if a user sets the default permission for geolocation or indexedDB storage to "Block" for all sites, setting it back to "Always Ask" won't work, so those features will be permanently disabled unless their corresponding prefs are manually changed in about:config.

Comment 5

[Dão Gottwald [:dao]]

http://hg.mozilla.org/mozilla-central/rev/7842f8720312

Comment 6

[:Margaret Leibovic]

Pushed to mozilla-aurora:
http://hg.mozilla.org/releases/mozilla-aurora/rev/5fe8136656ef

Comment 7

[Virgil Dicu [:virgil] [QA]]

Could anyone please provide some simple STR in order to completely verify this issue?

Comment 8

[:Margaret Leibovic]

(In reply to Virgil Dicu from comment #7)
> Could anyone please provide some simple STR in order to completely verify
> this issue?

In about:permissions, try changing the geolocation or indexedDB permissions for "All Sites", then go to a website that tries to use these features, and verify that Firefox asks/blocks as expected.

Comment 9

[Virgil Dicu [:virgil] [QA]]

Mozilla/5.0 (Windows NT 6.1; rv:7.0) Gecko/20100101 Firefox/7.0
Mozilla/5.0 (X11; Linux x86_64; rv:7.0) Gecko/20100101 Firefox/7.0
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:7.0) Gecko/20100101 Firefox/7.0
Mozilla/5.0 (Windows NT 5.1; rv:7.0) Gecko/20100101 Firefox/7.0

Verified fixed on Ubuntu 11.04, Mac OS 10.6, Windows XP and Windows 7.

Checked using the following site: http://html5demos.com/geo

Firefox acts consistent when toggling the settings in about:permissions.