213445 - XSLT form elements radio button - html/xml pages don't match

Status: RESOLVED FIXED

(Core :: Layout: Form Controls, defect)

Description

[rnorberg]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.4) Gecko/20030624
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.4) Gecko/20030624

XSLT 1.4 Test rendering of form elements

Reproducible: Always

Steps to Reproduce:
Steps to Reproduce:
1.open browser
2.open forms.html in one tab
3.open forms.xml in another tab
4.compare the two tabs
Actual Results:  
#2 radio button directly under text "Unchecked checkbox" doesn't match :
        it is "checked" on the html page
        it is "unchecked" on the xml page

Expected Results:  
html page and output of the xml page should match pixel for pixel

Comment 1

[rnorberg]

Attachment: xslt form testcase

Comment 2

[rnorberg]

Attachment: xslt form testcase

Comment 3

[rnorberg]

Attachment: stylesheet

Comment 4

[rnorberg]

You must download the files to your local machine before testing.
Download forms.html, forms.xml, and indentity.xsl to c:\temp.
The forms.xxx files need the stylesheet - identity.xsl.
Then open the forms.xxx files in your mozilla browser.

Comment 5

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

The problem is that having two different forms with radiogroups of same name
confuses the code. This isn't XSLT specific but also happens in XHTML. Testcase
comming up

Comment 6

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

Attachment: minimal testcase

Comment 7

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

Attachment: patch v1

This patch makes us treat not having a parent as not being in the document.
Basically it makes us deal with the case of mDocument being set and mParent
being null, which only happens during parsing.

Comment 8

[Peter Van der Beken [:peterv]]

Giving to sicking since he seems to have been working on this.

Comment 9

[Johnny Stenback (:jst)]

Comment on attachment 128287 [details] [diff] [review]
patch v1

sr=jst

Comment 10

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

Checked in, thanks for reviews