102560 - get rid of the extra copies of contents.rdf (that we added for mapi) because it will be a maintainence nightmare.

Status: VERIFIED DUPLICATE of bug 112724

(MailNews Core :: Simple MAPI, defect, P3)

Description

[(not reading, please use [e-mail] instead)]

as soon as racham lands his Windows XP integration bug fix (#97650)
we can fix how we integrate with Simple MAPI.

I think we can fix it to use nsIMessengerOSIntegration.idl / 
nsMessengerWinIntegration.cpp

(we might have to extend nsIMessengerOSIntegration.idl to do what we need.)

fixing this would mean revisiting the UI fixes for #95122 and #100668

I'd like to figure a way around having multiple content.rdf files and other 
issues with the current approach to mapi integration. 

when I figure it out, I can hand this bug back to srilatha.

I'm hoping we can

Comment 1

[(not reading, please use [e-mail] instead)]

I'm hoping we can...

get rid of the 4 copies of contents.rdf, which will be a maintainence nightmare.

I'd also like to remove the mapi code from accountUtils.js and move it into 
nsMessengerWinIntegration.cpp.

part of this will mean moving code from js to C++, but that should not be hard.

Comment 2

[(not reading, please use [e-mail] instead)]

good clean up for mapi, 0.9.6

Comment 3

[Jaime Rodriguez, Jr.]

If this is not needed to land Bug 102570 for the branch, we should remove the 
dependency from the branch landing bug ... just a suggestion.

Comment 4

[Jaime Rodriguez, Jr.]

Marking nsbranch-, so it hits the radar for the next release.

Comment 5

[Tiantian Kong]

Mark as depedent to meta bug 103807

Comment 6

[scottputterman]

moving to 1.0.  Feel free to take this one when you want but it doesn't sound
like it would hold 0.9.6 or 0.9.7 unless it prevented Mapi from getting in.

Comment 7

[(not reading, please use [e-mail] instead)]

mscott's figured out how to get rid of the extra copies of contents.rdf

let me morph this bug into that, and re-assign to him.

thanks mscott!

Comment 8

[(not reading, please use [e-mail] instead)]

over to mscott.

Comment 9

[Scott MacGregor]

I forgot Seth already gave me this bug. I actually just created a new bug for
this and put a patch in it. Please see 112724 for the mozilla changes and
bugscape bug #11194 for the commercial changes.

*** This bug has been marked as a duplicate of 112724 ***

Comment 10

[Trix Supremo]

verified dupe