Created attachment 324407 [details] [diff] [review] fix Another tiny step toward saner bookmark code. This moves the logic for how the top-10 list is actually decided out of the base bookmark classes, and into the top-10-managing class where it belongs (and re-expresses it in terms of sort descriptors). It also fixes a bug where the list may not be calculated correctly when two bookmarks have the same URL but different visit counts. More sort cleanup will follow, but I want to keep the pieces small.
Comment on attachment 324407 [details] [diff] [review] fix sr=pink
Landed on trunk.