41772 - No new window with show="new" or target="_blank" with href XML file

Status: VERIFIED FIXED

(Core :: XML, defect, P3)

Description

[Heikki Toivonen (remove -bugzilla when emailing directly)]

I think this worked, so it is a regression. I have only tested with NT so 
leaving OS and Platform as NT/PC.

Comment 1

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Added chrisd to CC (chrisd found this).

After a bit more testing it appears that the fault is not at XLink, at least not 
completely. What matters most is in what format the target file is in.

I will attach two test files.

1. HTML with 4 links
   a) URL points to XML, target="_blank"
      * opens in current window, BUG 1
   b) URL points to XML, target="_new"
      * does not open anything, BUG 2
   c) URL points to HTML, target="_blank"
      * opens OK
   d) URL points to HTML, target="_new"
      * opens OK

2. XML with 2 links
   a) URL points to XML, show="new"
      * opens in current window, BUG 1
   b) URL points to HTML, show="new"
      * opens OK

I tested the HTML file with NS6PR1 as well. Same results as with the latest 
Mozilla (just pulled and built).

Comment 2

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Attachment: HTML with 4 links

Comment 3

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Attachment: XML with 2 links

Comment 4

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Attachment: Fix target="_new" to work as target="_blank"

Comment 5

[Heikki Toivonen (remove -bugzilla when emailing directly)]

M17 is going to be cut earlier than I thought.

Comment 6

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Nominated for nsbeta3. It is really confusing if the functionality of the link 
depends on the type of the target file.

Comment 7

[Nisheeth Ranjan]

This is confusing behavior and something that a lot of people who use simple 
XLinks will run into.  Marking nsbeta3+ and ccing myself.

Comment 8

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Attachment: FIX BUG 1: Somebody had forgot to register text/xml content handler

Comment 9

[Heikki Toivonen (remove -bugzilla when emailing directly)]

r=jst

Comment 10

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Fix checked in.

Comment 11

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Doh, my checkin 08/10/2000 15:07 actually fixed bug 47930, bogus bug number in 
checkin report. But in any case, both this and the aforementioned bugs have been 
fixed.

Comment 12

[Chris Petersen]

Verified fix in Aug 24th build.