Due to someone *ahem* ignoring my review requests all week, I've collected a large set of bookmarks changes, including: - renaming create[Folder|Bookmark]Ex functions to create[*]WithDetails, and updating callers. - persistence of sort data in bookmarks list views across all bookmarks list views (so sorting one view sorts all) - persistence of column width and visibility data across sessions - remove existing dynamic IE Favorites Folder if we do a one-off import of IE Favorites at start up (to avoid having two Imported IE Favorites folders). - Removes underlined appearance of links in sidebar. - Applies uniform single click folder opening behaviour to History panel. Patch to be attached.

*** Bug 128813 has been marked as a duplicate of this bug. ***

Comment on attachment 72388 [details] [diff] [review] patch to fix described issues r=blake

nsbeta1+ per Nav triage team, due to fixes for plussed bugs, but we certainly don't *need* to change function names for MachV.

hyatt says "sr=hyatt"

Comment on attachment 72388 [details] [diff] [review] patch to fix described issues a=asa (on behalf of drivers) for checkin to the 1.0 trunk

checked in.

I am seeing a reproducible crash in nsXULOutlinerBuilder::GetTemplateActionCellFor when switching to bookmarks tab in sidebar (under certain conditions) - see bug 131247. Could that be caused by this check-in?

VERIFIED Fixed with 20020401 builds