Add entry for allow/disable/remember geolocation to Permissions tab in Page Info

RESOLVED FIXED in seamonkey2.0b2

Status

SeaMonkey
Page Info
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: Robert Kaiser, Assigned: Philip Chee)

Tracking

Trunk
seamonkey2.0b2
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
We should add allow/disable/remember geolocation to the Permissions tab in Page Info, just as Firefox did in bug 491732. The conversion from content pref to permissions service in there sounds like a good idea as well.
(Reporter)

Updated

8 years ago
Depends on: 491732
(Assignee)

Updated

8 years ago
Blocks: 467530
(Assignee)

Comment 1

8 years ago
Created attachment 388020 [details] [diff] [review]
Patch v1.0 straight port
[Checkin: Comment 7]

Taking bug.

> The conversion from content pref to
> permissions service in there sounds like a good idea as well.

Appears to have been fixed in some other bug.

> We should add allow/disable/remember geolocation to the Permissions tab in Page
> Info, just as Firefox did in bug 491732.

I tested this against google maps. The Google geolocation service found the right country but the wrong state and placed me right in the middle of a major highway, but I guess that's a google bug not ours.
Assignee: nobody → philip.chee
Status: NEW → ASSIGNED
Attachment #388020 - Flags: superreview?(neil)
Attachment #388020 - Flags: review?(neil)
(Reporter)

Comment 2

8 years ago
(In reply to comment #1)
> > The conversion from content pref to
> > permissions service in there sounds like a good idea as well.
> 
> Appears to have been fixed in some other bug.

Yes, we pulled, that into the the other one.

> > We should add allow/disable/remember geolocation to the Permissions tab in Page
> > Info, just as Firefox did in bug 491732.
> 
> I tested this against google maps. The Google geolocation service found the
> right country but the wrong state and placed me right in the middle of a major
> highway, but I guess that's a google bug not ours.

Yes, what location you get is completely up to Google.

Comment 3

8 years ago
Comment on attachment 388020 [details] [diff] [review]
Patch v1.0 straight port
[Checkin: Comment 7]

Firefox are getting good at this ;-)
Attachment #388020 - Flags: superreview?(neil)
Attachment #388020 - Flags: superreview+
Attachment #388020 - Flags: review?(neil)
Attachment #388020 - Flags: review+
(Assignee)

Comment 4

8 years ago
> Firefox are getting good at this ;-)
Well I did fix the broken ARIA/focus stuff that I fixed in the rest of our Permissions tab, and that is still broken in Firefox ;-)
(In reply to comment #4)
> > Firefox are getting good at this ;-)
> Well I did fix the broken ARIA/focus stuff that I fixed in the rest of our
> Permissions tab, and that is still broken in Firefox ;-)

Have you filed a bug?
(Assignee)

Comment 6

8 years ago
In reply to comment #5)
> Have you filed a bug?

Filed Bug 504053
(Assignee)

Updated

8 years ago
Keywords: checkin-needed
(Assignee)

Updated

8 years ago
Attachment #388020 - Attachment description: Patch v1.0 straight port. → [for checkin] Patch v1.0 straight port.
Comment on attachment 388020 [details] [diff] [review]
Patch v1.0 straight port
[Checkin: Comment 7]


http://hg.mozilla.org/comm-central/rev/16d40cae21eb
Attachment #388020 - Attachment description: [for checkin] Patch v1.0 straight port. → Patch v1.0 straight port [Checkin: Comment 7]
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.0b2
No longer blocks: 467530
You need to log in before you can comment on or make changes to this bug.