Bug 203756 added the browser.urlbar.autoFill pref for auto-filling the best match in the address bar. However, the pref was never added to firefox.js, so it doesn't show up in about:config and must be newly added. This bug seeks to add the pref, defaulted to off so as not to change behavior. Bug 186136 has more discussion about broader issues such as fixing autoFill to suck less. All I'm trying to do here is make the pref visible.
Created attachment 219196 [details] [diff] [review] Fix for firefox.js Add the pref to firefox.js. Since this is part of browser, it doesn't belong in all.js.
Attachment #219196 - Flags: review?
Comment on attachment 219196 [details] [diff] [review] Fix for firefox.js Tried to set this to get review from Ben Goodger, didn't seem to work. Trying again.
Attachment #219196 - Flags: review? → review?(bugs)
Comment on attachment 219196 [details] [diff] [review] Fix for firefox.js As a general note, there is no requirement to ensure prefs show up in about:config, but this one is not rare, though its far from commonly sought. The dependency isn't necessary, but meh.
Attachment #219196 - Flags: review?(bugs) → review+
Peter: do you want me to check this in?
Yes please, since I don't have CVS privileges. It would be nice to get this onto the 1.8 branch as well.
Status: NEW → RESOLVED
Last Resolved: 13 years ago
OS: Linux → All
Hardware: PC → All
Resolution: --- → FIXED
Target Milestone: --- → Firefox 2 alpha2
Version: unspecified → 2.0 Branch
Checked in on the 1.8 branch too. mozilla/browser/app/profile/firefox.js 18.104.22.168
You need to log in before you can comment on or make changes to this bug.