Last Comment Bug 580663 - Remove old bookmarks system code
: Remove old bookmarks system code
Product: SeaMonkey
Classification: Client Software
Component: Bookmarks & History (show other bugs)
: Trunk
: All All
: -- normal (vote)
: seamonkey2.1a3
Assigned To: Robert Kaiser
Depends on: 580656 580658 580660 580662
Blocks: SMPlacesBMarks 624687
  Show dependency treegraph
Reported: 2010-07-21 09:16 PDT by Robert Kaiser
Modified: 2011-01-11 04:07 PST (History)
2 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---

v2: remove old system (423.93 KB, patch)
2010-07-21 09:48 PDT, Robert Kaiser
iann_bugzilla: review+
Details | Diff | Splinter Review

Description Robert Kaiser 2010-07-21 09:16:52 PDT
Once places bookmarks have landed, we should remove the old bookmarks system.

This is a split-off of part 6 of bug 498596, for easier review (as now that
part is isolated here).
Comment 1 Robert Kaiser 2010-07-21 09:48:07 PDT
Created attachment 459045 [details] [diff] [review]
v2: remove old system

This is the most stable patch of all those I have for places bookmarks, and it should not be that hard to review, even if it's quite large.
Comment 2 Robert Kaiser 2010-07-28 10:45:15 PDT
Comment on attachment 459045 [details] [diff] [review]
v2: remove old system

Requesting additional review from Ian on all patches but the actual core - I'll take reviews from whoever gets around to it first.
Comment 3 Ian Neal 2010-08-01 09:11:52 PDT
Comment on attachment 459045 [details] [diff] [review]
v2: remove old system

I think you have caught everything apart from those entities mentioned in my comments on bug 580658.
Comment 4 Robert Kaiser 2010-08-08 13:12:26 PDT
Pushed as

Note You need to log in before you can comment on or make changes to this bug.