Closed Bug 406425 Opened 17 years ago Closed 15 years ago

Limit how many adaptive input history entries to track

Categories

(Firefox :: Address Bar, defect)

Product:
Firefox
Component:
Address Bar
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED WONTFIX

People

(Reporter: Mardak, Assigned: Mardak)

References

Details

Attachments

(1 file)

v1
1.57 KB, patch
Details | Diff | Splinter Review 
Potentially moz_inputhistory could get big and might affect performance.

What would be a good number of entries to track?

I've been using the adaptive bar for almost a month and I have barely 80 entries.. and quite a few of those I wouldn't really care if they were removed. (But my browsing habits are probably different from others :p)
Attached patch v1 — — Splinter Review 
How about 200? ;) Do we need a pref for this?
Assignee: nobody → edilee
Status: NEW → ASSIGNED
IMO, it's always nice to have a pref for settings which may have a performance impact.
Aren't we pretty aggressive about reducing and expiring these?  I'm not sure we need a hard cap, so is this still valid?
Yeah we decay things and remove entries that have decayed past usefulness, so we won't need this.
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: