157623 - Clicking on a JS link in a mail message will open the source in the preview pane.

Status: VERIFIED FIXED

(MailNews Core :: Networking, defect)

Description

[[e-mail] (gone - use [e-mail] instead)]

Build ID: 7/15/2002, commercial 1.0 Branch build.

I'm not sure this should go into the Security component, but for now...

Summary: Clicking on a JS link in a mail message will open the source in the
preview pane.

Steps to Reproduce:

1.  Send yourself a message with a link contained in it such as,
'http://people.netscape.com/stephend/js.html'.
2.  Read the message, and click on the link.

Expected Results:

I would think we'd want to open this in the browser window.

Actual Results:

We open this in the mail preview pane, (overriding all of the message's other
contents, too).

Comment 1

[David :Bienvenu]

that's weird - I would have thought that would be an html link.

Comment 2

[[e-mail] (gone - use [e-mail] instead)]

Sorry, yes, I meant - http://people.netscape.com/stephend/adec/dashboard.js

Comment 3

[[e-mail] (gone - use [e-mail] instead)]

Sorry, this really isn't security.  I'm going to stick this in
Networking:General, simply because I'm not sure where the content-type stuff lives.

Comment 4

[Boris Zbarsky [:bzbarsky]]

So.... I can't find where the content listener for them mailwindow actually
draws a distinction between content types... it seems to mark all types as not
preferred and defers to the docshell for whether it can handle them....

Comment 5

[[e-mail] (gone - use [e-mail] instead)]

nsbeta1, because I want Buffy to get this right.

Comment 6

[Samir Gehani]

Mail triage team: nsbeta1+/adt2

Comment 7

[[e-mail] (gone - use [e-mail] instead)]

New url for testing: http://people.netscape.com/stephend/dashboard.js

Comment 8

[[e-mail] (gone - use [e-mail] instead)]

-> seth

Comment 9

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

I think a recent change of mine for bug #199360 might have fixed this.

double checking...

Comment 10

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

ok, before my fix, I get the "open / save / unknown content handler dialog, for
type application/x-javascript.

after my fix, because that's a http url, we do load it in a browser window, and
I see the source for the js in the browser window.

either seem ok to me.  neither are what stephend described back on 7/15/2002, so
something else must have fixed it.

but since there is a change to the scenario, do to bug #199360, I'm going to
claim victory and mark this fixed.

esther / stephend, if it should be wfm, let me know.

Comment 11

[[e-mail] (gone - use [e-mail] instead)]

Seth / Esther, yup this is fixed in the 2003-04-23-04 trunk WinXP/Windows 2000
builds.  Thanks!

/me waves to his old Mail team ;-)

I'll let Esther do the multi-platform legwork...

Comment 12

[esther]

Using trunk builds 20030509 on linux and macosx and winxp I sent myself an email
with the link in comment #7, opened it and clicked on the link.  The link opened
in a browser window.  Verified.