bugzilla.mozilla.org has resumed normal operation. Attachments prior to 2014 will be unavailable for a few days. This is tracked in Bug 1475801.
Please report any other irregularities here.

Expired visits unnecessarily reset frecencies to -1

RESOLVED FIXED in Firefox 3 beta4

Status

()

Firefox
Address Bar
RESOLVED FIXED
11 years ago
8 years ago

People

(Reporter: Mardak, Assigned: Mardak)

Tracking

Trunk
Firefox 3 beta4
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

11 years ago
When we expire visits for a page, we reset the frecency to -1 because potentially we cleared out all its visits. But we don't need to be so aggressive in resetting because we can check if the place still has visits and/or bookmarks. If so, we can reuse the frecency as a better estimate than just -1.

Patch is part of bug 417810.
(Assignee)

Comment 1

11 years ago
This is one cause for bug 417810 which is blocking+
Flags: blocking-firefox3?
(Assignee)

Comment 2

11 years ago
Created attachment 304750 [details] [diff] [review]
v1
Assignee: nobody → edilee
Status: NEW → ASSIGNED
Attachment #304750 - Flags: review?(dietrich)
Comment on attachment 304750 [details] [diff] [review]
v1

r=me, thanks
Attachment #304750 - Flags: review?(dietrich) → review+
(Assignee)

Comment 4

11 years ago
Checking in toolkit/components/places/src/nsNavHistoryExpire.cpp;
/cvsroot/mozilla/toolkit/components/places/src/nsNavHistoryExpire.cpp,v  <--  nsNavHistoryExpire.cpp
new revision: 1.41; previous revision: 1.40
done
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Flags: blocking-firefox3?
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3 beta4
You need to log in before you can comment on or make changes to this bug.