Closed
Bug 436677
Opened 17 years ago
Closed 17 years ago
Clean up rdf consumers of the addressbook
Categories
(Thunderbird :: Address Book, defect)
Thunderbird
Address Book
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3
People
(Reporter: jminta, Assigned: jminta)
References
Details
(Whiteboard: [T.M.=Thunderbird 3.0a2])
Attachments
(1 file)
2.76 KB,
patch
|
mkmelin
:
review+
|
Details | Diff | Splinter Review |
Now that nsIAbManager has a directories attribute, there's very little reason for anyone to be using the rdf service to deal with addressbook directories. This patch switches a couple of js consumers I found that were still using the rdf service.
Attachment #323211 -
Flags: review?(mkmelin+mozilla)
Comment 1•17 years ago
|
||
Comment on attachment 323211 [details] [diff] [review]
patch v1
[Checkin: Comment 2]
Looks good, Joey!
Attachment #323211 -
Flags: review?(mkmelin+mozilla) → review+
Updated•17 years ago
|
Assignee: nobody → jminta
Target Milestone: --- → Thunderbird 3
Assignee | ||
Comment 2•17 years ago
|
||
Patch checked in. Regression tests should check that the address-book list in Composition preferences is properly populated, and that address-book options in the message-preview pane don't offer to duplicate known addresses.
Checking in mail/base/content/msgHdrViewOverlay.js;
/cvsroot/mozilla/mail/base/content/msgHdrViewOverlay.js,v <-- msgHdrViewOverlay.js
new revision: 1.104; previous revision: 1.103
done
Checking in mail/components/preferences/compose.js;
/cvsroot/mozilla/mail/components/preferences/compose.js,v <-- compose.js
new revision: 1.21; previous revision: 1.20
done
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Comment 3•16 years ago
|
||
Comment on attachment 323211 [details] [diff] [review]
patch v1
[Checkin: Comment 2]
Ftr,
it looks like SeaMonkey did not have these codes.
Updated•15 years ago
|
Attachment #323211 -
Attachment description: patch v1 → patch v1
[Checkin: Comment 2]
Updated•15 years ago
|
Whiteboard: [T.M.=Thunderbird 3.0a2]
You need to log in
before you can comment on or make changes to this bug.
Description
•