12117 - REMINDER: Review URL auto detection in libmime

Status: VERIFIED FIXED

(MailNews Core :: MIME, defect, P3)

Description

[rhp (gone)]

Do final review before production release.

Comment 1

[rhp (gone)]

Need to deal with this after sabbatical.

- rhp

Comment 2

[lchiang]

(target milestone is M11 or M12 - add to mail beta tracking bug)

Comment 3

[rhp (gone)]

Actually took care of this one before I left on sabbatical. For details, send
me an email.

- rhp

Comment 4

[ppandit]

Rich -

I can't find within CVS Query Tool which of your changes are involved with this 
bug. Can you tell me how to verify or show me the code changes so I can visually 
verify.

Thx,
Par

Comment 5

[Ben Bucksch (:BenB)]

I have no idea, what this bug is about. Should I?
ppandit: FYI: I wrote the current URL detection code.

Comment 6

[ppandit]

I am verifying this bug based on

1) The code has been rewritten by another developer (BenB)
2) This bug was about being able to detect instant messenger URLs
3) This bug is old and no one has complained about it or updated it

Please reopen if needed.

Comment 7

[ppandit]

That didn't take long. After emailing with Ben, I am going to reopen and let 
the developers mark this fixed after they are sure that AIM urls actually work.

Comment 8

[rhp (gone)]

This has been fixed by a general solution for handling URL detection. I don't 
think there is an AIM url handler and I don't want this on my bug list while I 
wait for one to be developed. 

- rhp

Comment 9

[ppandit]

Rich - is there a testcase for this? Are all issues resolved?

Par

Comment 10

[rhp (gone)]

This was an internal thing that has been taken care of. :-)

- rhp

Comment 11

[ppandit]

Based on rhp comments, marking as verified