User:Sspitzer/GlobalFrecency: Difference between revisions

From MozillaWiki
Jump to navigation Jump to search
(New page: notes: 1) unvisited bookmarks are no longer hidden, but unvisited livemark items are. this happens in nsNavBookmarks::InsertBookmark(), where we call UpdateFrecencyAndHiddenForBookmark(...)
 
No edit summary
Line 4: Line 4:


unvisited bookmarks are no longer hidden, but unvisited livemark items are.  this happens in nsNavBookmarks::InsertBookmark(), where we call UpdateFrecencyAndHiddenForBookmark(), but only if the parent is not a livemark
unvisited bookmarks are no longer hidden, but unvisited livemark items are.  this happens in nsNavBookmarks::InsertBookmark(), where we call UpdateFrecencyAndHiddenForBookmark(), but only if the parent is not a livemark
2)
when you remove a bookmark, should the frecency change for that place?  yes, it should, as we gave weight to bookmarks.  RemoveItem() calls UpdateBookmarkHashOnRemove() with the place id, so we do this work there.  Note, if you remove a folder, we will eventually call RemoveItem(), so that's how this works with removing folders.

Revision as of 03:48, 28 December 2007

notes:

1)

unvisited bookmarks are no longer hidden, but unvisited livemark items are. this happens in nsNavBookmarks::InsertBookmark(), where we call UpdateFrecencyAndHiddenForBookmark(), but only if the parent is not a livemark

2)

when you remove a bookmark, should the frecency change for that place? yes, it should, as we gave weight to bookmarks. RemoveItem() calls UpdateBookmarkHashOnRemove() with the place id, so we do this work there. Note, if you remove a folder, we will eventually call RemoveItem(), so that's how this works with removing folders.