Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Replace %APP% with firefox in firefox preferences

RESOLVED FIXED in Firefox 6

Status

()

Firefox
Preferences
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: glandium, Assigned: glandium)

Tracking

unspecified
Firefox 6
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Assignee)

Description

6 years ago
Created attachment 526227 [details] [diff] [review]
Patch

This is something I have been doing for a while on Debian, where %APP% resolves to iceweasel, yet hadn't bothered filing a bug until now that I saw bug 602367 comment 15:

>> Why aren't we using '?src=%APP%' here?
>
> It's in the Firefox prefs so I don't think there is much to be gained from
> changing that now.

With the same rationale, it should be safe to replace %APP% with firefox in the same file.
Attachment #526227 - Flags: review?(dtownsend)
Comment on attachment 526227 [details] [diff] [review]
Patch

My comment was more because we were pushed for time so I didn't think it worth bothering with, not necessarily that we should do it everywhere but I'll let one of the browser peers make that call.
Attachment #526227 - Flags: review?(dtownsend) → review?(gavin.sharp)
Attachment #526227 - Flags: review?(gavin.sharp) → review+
(Assignee)

Comment 2

6 years ago
Created attachment 530986 [details] [diff] [review]
Replace %APP% with "firefox" in Firefox preferences.

As landed
(Assignee)

Comment 3

6 years ago
http://hg.mozilla.org/mozilla-central/rev/9e31df64bfd7
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 6

Updated

6 years ago
Assignee: nobody → mh+mozilla
You need to log in before you can comment on or make changes to this bug.