48217 - Context menu item "Show in new window" does not use XML Base for XLink

Status: VERIFIED FIXED

(Core :: XML, defect, P3)

Description

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

The Show in new window menu item only grabs the href of the simple XLink and 
does not use XML Base nor HTML BASE.

There are two immediate solution and one long term solution:
1) Duplicate the C++ code in JS that makes use of XML Base (the C++
   code gets executed with show="new" aatribute when you click the
   link normally). Slow/bad. This is standards compliant.
2) Create a function thruogh which the JS code can get the the XML Base
   computed by C++ code. Fast. Not standards compliant.
3) DOM3 is going to address XML Base. Wait for what DOM3 says.

Comment 1

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

Since XML is still pretty rare on the web and XML Base even more so, I am 
inclined to Future this bug. I want to fix this by using the Base property that 
will hopefully make it into DOM3.

This bug has been marked "future" because the original netscape engineer working 
on this is over-burdened. If you feel this is an error, that you or another
known resource will be working on this bug,or if it blocks your work in some way 
-- please attach your concern to the bug for reconsideration.

Comment 2

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

Attachment: Proposed fix 1

Comment 3

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

With the baseURI property this indeed becomes quite simple. I fixed some other
cases where we benefit from baseURI property as well. Are there others?

Comment 4

[Alec Flett]

this is awesome. adding sicking, because his stuff that I'm about to check in
would use this. Sicking - I'll check your stuff in anyway today, but I'm hoping
for an additional patch for using .baseURI in bug 1995

Comment 5

[Alec Flett]

by the way, sr=alecf :)

Comment 6

[Blake Ross]

cool. r=blake

Comment 7

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

Fix checked in.

Comment 8

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

Reopening 'cos baseURI got backed out. Will reassign to jst who can enable this
again once the DOM conversion branch lands.

Comment 9

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

Please enable this fix once the DOM conversion lands.

Comment 10

[Johnny Stenback (:jst)]

Fixed.

Comment 11

[Chris Petersen]

Changing QA contact. Xlink issue.

Comment 12

[bsharma]

Marking verified as per above developer comments.