Remove old bookmarks system code

RESOLVED FIXED in seamonkey2.1a3

Status

SeaMonkey
Bookmarks & History
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: Robert Kaiser, Assigned: Robert Kaiser)

Tracking

Trunk
seamonkey2.1a3
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
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).
(Assignee)

Comment 1

7 years ago
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.
Attachment #459045 - Flags: review?(neil)

Updated

7 years ago
Summary: Remove old bookmarks sytem code → Remove old bookmarks system code
(Assignee)

Comment 2

7 years ago
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.
Attachment #459045 - Flags: review?(iann_bugzilla)

Comment 3

7 years ago
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.
Attachment #459045 - Flags: review?(iann_bugzilla) → review+
(Assignee)

Updated

7 years ago
Attachment #459045 - Flags: review?(neil)
(Assignee)

Comment 4

7 years ago
Pushed as http://hg.mozilla.org/comm-central/rev/b7d14a8dd2d0
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.1a3
Version: unspecified → Trunk

Updated

6 years ago
Blocks: 624687
You need to log in before you can comment on or make changes to this bug.