Hookup geolocation prompt to fennec's notification prompt.

VERIFIED FIXED in fennec1.0m5

Status

Firefox for Android Graveyard
General
VERIFIED FIXED
10 years ago
9 years ago

People

(Reporter: dougt, Assigned: dougt)

Tracking

Trunk
fennec1.0m5
x86
Linux

Details

Attachments

(1 attachment)

(Assignee)

Description

10 years ago
Created attachment 328984 [details] [diff] [review]
patch v.1
(Assignee)

Comment 1

10 years ago
Comment on attachment 328984 [details] [diff] [review]
patch v.1

basically the same thing as FF.
Attachment #328984 - Flags: review?(mark.finkle)
(Assignee)

Updated

10 years ago
Depends on: 437948
Comment on attachment 328984 [details] [diff] [review]
patch v.1

>diff --git a/chrome/content/browser.js b/chrome/content/browser.js
>+
>+  setupGeolocationPrompt: function()
>+  {
>+    var geolocationService = Cc["@mozilla.org/geolocation/service;1"].getService(Ci.nsIGeolocationService);
>+    
>+    geolocationService.prompt = function(request) {
>+      
>+      var notificationBox = gBrowser.getNotificationBox();

        var notificationBox = Browser.getNotificationBox();

We don't have a gBrowser (and might not have one) so use "Browser" as shown.

>+        var message = bundle_browser.getFormattedString("geolocation.requestMessage", [request.requestingURI.spec]);      
>+        notificationBox.appendNotification(message,
>+                                           "geolocation",
>+                                           "chrome://browser/skin/Info.png",

This image does not exist in Fennec. We should get Madhava to make an image that fits the Fennec theme. Until then, just pass null.

r+ with those tweaks.
Attachment #328984 - Flags: review?(mark.finkle) → review+
(Assignee)

Comment 3

10 years ago
changeset:   66:0c6c257bd42c
tag:         tip
user:        dougt@dougt-ubuntu
date:        Wed Jul 16 17:07:43 2008 -0700
summary:     Geolocation UI. Bug 444665. r=mfinkle
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
verified FIXED (using share, more info and don't share) on builds:

Mozilla/5.0 (X11; U; Linux armv6l; en-US; rv:1.9.3a1pre) Gecko/20090821
Fennec/1.0b3pre

and

Mozilla/5.0 (Macintosh; U; Intel Mac OSX 10.5; en-US; rv:1.9.2a2pre)
Gecko/20090808 Fennec/1.0b3pre
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.