Every bookmark registers for URL load notifications

RESOLVED FIXED in Camino1.5

Status

defect
P3
normal
RESOLVED FIXED
15 years ago
14 years ago

People

(Reporter: sfraser_bugs, Assigned: sfraser_bugs)

Tracking

({perf})

unspecified
Camino1.5
PowerPC
All
Dependency tree / graph

Details

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?
Assignee

Comment 2

15 years ago
Taking.
Assignee: pinkerton → sfraser_bugs
Assignee

Updated

14 years ago
Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → Camino1.1
Assignee

Updated

14 years ago
Blocks: 236373
Assignee

Comment 3

14 years ago
Fixed now; the bookmarks manager does the load listening, and forwards out to
the relevant bookmarks.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.