REMINDER: Review URL auto detection in libmime

VERIFIED FIXED in M12

Status

P3
normal
VERIFIED FIXED
20 years ago
11 years ago

People

(Reporter: rhp, Assigned: rhp)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

20 years ago
Do final review before production release.
(Assignee)

Updated

20 years ago
Status: NEW → ASSIGNED
Target Milestone: M12
(Assignee)

Comment 1

20 years ago
Need to deal with this after sabbatical.

- rhp

Updated

20 years ago
OS: Windows NT → All
Hardware: PC → All

Updated

20 years ago
Blocks: 11091

Comment 2

20 years ago
(target milestone is M11 or M12 - add to mail beta tracking bug)
(Assignee)

Updated

20 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 20 years ago
Resolution: --- → FIXED
(Assignee)

Comment 3

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

- rhp

Updated

19 years ago
QA Contact: lchiang → ppandit

Comment 4

19 years ago
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
I have no idea, what this bug is about. Should I?
ppandit: FYI: I wrote the current URL detection code.

Comment 6

19 years ago
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.
Status: RESOLVED → VERIFIED

Comment 7

19 years ago
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.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 8

19 years ago
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
Status: REOPENED → RESOLVED
Last Resolved: 20 years ago19 years ago
Resolution: --- → FIXED

Comment 9

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

Par
(Assignee)

Comment 10

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

- rhp

Comment 11

19 years ago
Based on rhp comments, marking as verified
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.