57437 - Links to anchors in external XML documents do not work

Status: VERIFIED FIXED

(Core :: XML, defect, P3)

Description

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

IE5 can follow these links (except the last one because it does not recognize ID
type attributes), but we can't (try clicking these URLs from here, or write them
in URLbar):

http://bugzilla.mozilla.org/showattachment.cgi?attach_id=16714#id1
http://bugzilla.mozilla.org/showattachment.cgi?attach_id=16714#id2
http://bugzilla.mozilla.org/showattachment.cgi?attach_id=16714#id3
http://bugzilla.mozilla.org/showattachment.cgi?attach_id=16714#id4

Look at bug 56285 for how this was fixed fro HTML files.

Comment 1

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

Nominating for RTM, this is basic functionality, and we can use the ideas from
bug 56285.

Comment 2

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

Attachment: Fix: Copy the way we do this in HTML content sink

Comment 3

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

Attachment: Use nsString for mRef instead of nsString*

Comment 4

[harishd]

sr=vidur for the second patch. It would be nice and fairly safe to fix this for 
XML/XHTML.

Comment 5

[Adam Lock]

Patch works great for me too. r=[e-mail]

Comment 6

[lchiang]

nisheeth - if you want PDT to look at this after you've gotten approval and
super reviews, you have to mark [rtm+] in the status whiteboard.  I couldn't
tell from your comments if you meant to do this or if you wanted adam to do this.

Comment 7

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

I am stealing this bug, I hope Adam won't mind...

Comment 8

[Nisheeth Ranjan]

Marking rtm- because this isn't a ship stopper.  The second patch can definitely
be checked into the trunk, though.

Comment 9

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

Fixed on TRUNK ONLY, will not go into branch.

Comment 10

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

*** Bug 55008 has been marked as a duplicate of this bug. ***

Comment 11

[Loco]

Test case from Duplicate bug:

http://bugzilla.mozilla.org/showattachment.cgi?attach_id=16024

Comment 12

[Loco]

Also including info from bug: (tested by reporter on M17, M18, and RTM builds)

Reproducible: Always
Steps to Reproduce:
1.Create an HTML frameset containing 2 frames
2.On the LHS use and HTML frame to containing links to locations within an XML
document.
3. On the RHS create an XML document using the HTML namespace for defining
anchor points within the XML document.

Actual Results:  With IE5 you can use the LH frame to find locations within the
RH XML document. With Mozilla it only finds the document, but not the location
within the document.

Expected Results:  It should have navigated to the correct position within the
XML document.

Comment 13

[Chris Petersen]

Marking verified in the 2001010209 build.

Comment 14

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

*** Bug 45338 has been marked as a duplicate of this bug. ***