Every bookmark registers for URL load notifications

RESOLVED FIXED in Camino1.5

Status

Camino Graveyard
Bookmarks
P3
normal
RESOLVED FIXED
13 years ago
13 years ago

People

(Reporter: Simon Fraser, Assigned: Simon Fraser)

Tracking

({perf})

unspecified
Camino1.5
PowerPC
All

Details

(Assignee)

Description

13 years ago
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

13 years ago
Taking.
Assignee: pinkerton → sfraser_bugs
(Assignee)

Updated

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

Updated

13 years ago
Blocks: 236373
(Assignee)

Comment 3

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