256514 - (xsltlinks) Links don't work in documents created via XSLT

Status: VERIFIED FIXED

(Core :: XSLT, defect)

Description

[Doug Wright]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.2) Gecko/20040820 Firefox/0.9.1+
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.2) Gecko/20040820 Firefox/0.9.1+

Quick example is Dean Edward's page getting XHTML documents to display in IE. A
completely different example is http://www.dougsdvds.info/list/test.xml. 

This WORKS in 1.7.2
This WORKS in 1.7.3(20040822 nightly)
This DOESN'T WORK in Firefox 0.9.1+ (20040820 nightly)

I've filed it here, because there doesn't seem to be an appropriate section
under the Firefox heading

Reproducible: Always
Steps to Reproduce:
1. Load a document created via XSLT
2. Try and click a link


Actual Results:  
Page didn't change, status bar didn't show target

Expected Results:  
Followed the link

Comment 1

[Doug Wright]

Forgot to mention - it worked in Firefox builds around 20040725 (roughly - maybe
a week before), cos that's when I created the 2nd page mentioned above. I can't
narrow it down any further I'm afraid.

Comment 2

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

A smaller testcase would be nice. And please attach it to the bug rather then
just point to a url. URLs tend to go away after a while.

Comment 3

[Doug Wright]

Attachment: XSL stylesheet

Comment 4

[Doug Wright]

Attachment: XML

Comment 5

[Doug Wright]

View the 2nd attachment to see the problem.

Comment 6

[Axel Hecht]

Odd stuff happening.
Middle click opens the link in a new tab, context menu works as well. Just
single click doesn't work. My debug aviary doesn't complain about anything, though.

Comment 7

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

This sounds pretty bad and a major regression. It would be pretty useless to
ship xslt support with this.

Comment 8

[Doug Wright]

I've tested various nightly builds, and have managed to establish that this
regressed between the 17th and the 19th. The build for the 18th crashes on
startup, so I can't test that.

The list of checkins between those builds is at http://tinyurl.com/3s9tj

This is as far as my skills take me I'm afraid.

Comment 9

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Maybe the fix for bug 198254?

http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&subdir=mozilla/layout/html/base/src&command=DIFF_FRAMESET&file=nsPresShell.cpp&rev1=3.712.2.2.2.3&rev2=3.712.2.2.2.4&root=/cvsroot

Comment 10

[Axel Hecht]

(In reply to comment #9)
> Maybe the fix for bug 198254?
> 
>
http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&subdir=mozilla/layout/html/base/src&command=DIFF_FRAMESET&file=nsPresShell.cpp&rev1=3.712.2.2.2.3&rev2=3.712.2.2.2.4&root=/cvsroot

backed out locally and it works on yesterdays aviary.

Comment 11

[Peter Van der Beken [:peterv]]

Attachment: v1

Comment 12

[Johnny Stenback (:jst)]

Comment on attachment 156890 [details] [diff] [review]
v1

r+sr=jst

Comment 13

[Johnny Stenback (:jst)]

Comment on attachment 156890 [details] [diff] [review]
v1

This is a regression from a change that went in on both the aviary and 1.7
branches. We should take this regression fix on both branches IMO.

Comment 14

[Mike Kaply [:mkaply]]

Comment on attachment 156890 [details] [diff] [review]
v1

a=mkaply

Comment 15

[Johnny Stenback (:jst)]

Fix landed on the aviary and 1.7 brances.

Comment 16

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

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