Closed Bug 279994 Opened 20 years ago Closed 20 years ago

Every bookmark registers for URL load notifications

Categories

(Camino Graveyard :: Bookmarks, defect, P3)

Product:
Camino Graveyard
Component:
Bookmarks
Platform:
PowerPC
All
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
Camino1.5

People

(Reporter: sfraser_bugs, Assigned: sfraser_bugs)

References

Details

(Keywords: perf)

Currently every bookmark registers for URL load notifications, so on every URL load, the notification manage will fire off notifications to potentially hundreds of bookmarks. This is done to set last-visit dates (which we don't really use anywhere anyway). This sucks. If we really want to maintain last-visit, we should maintain a single observer of URL loads that has a map from URL to bookmark items.
isn't it also used for computing the top10 list visit counts?
Taking.
Assignee: pinkerton → sfraser_bugs
Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → Camino1.1
Blocks: 236373
Fixed now; the bookmarks manager does the load listening, and forwards out to the relevant bookmarks.
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.