bin/45859: atf-report - tests-results.xsl produces incorrect output

[paul%whooppee.com@localhost]

>Number:         45859
>Category:       bin
>Synopsis:       atf-report - tests-results.xsl produces incorrect output
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    bin-bug-people
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Fri Jan 20 03:15:00 +0000 2012
>Originator:     Paul Goyette
>Release:        NetBSD 5.99.60
>Organization:
-------------------------------------------------------------------------
| Paul Goyette     | PGP Key fingerprint:     | E-mail addresses:       |
| Customer Service | FA29 0E3B 35AF E8AE 6651 | paul at whooppee.com    |
| Network Engineer | 0786 F758 55DE 53BA 7731 | pgoyette at juniper.net |
| Kernel Developer |                          | pgoyette at netbsd.org  |
-------------------------------------------------------------------------
>Environment:
        
        
System: NetBSD 5.99.60 (GENERIC) #0: Fri Jan 20 00:31:20 UTC 2012 
root%screamer.whooppee.com@localhost:/test-bed/obj/sys/arch/amd64/compile/GENERIC
Architecture: x86_64
Machine: amd64
>Description:
        (Please reassign if bin is not the correct category for code that
        lives in src/external/ hierarchy.)

        The tests-results.xsl file as provided generates invalid .html code.  
        While the invalid tags are (apparently) ignored by browsers, they
        prevent right-margin text alignment for the "Duration" field as
        specified in the tests-results.css style sheet.
>How-To-Repeat:
        1. Use "atf-run | atf-report -o xml:xxx.xml" to generate the xml
           output.  The file will contain text similar to the following for
           the test-program and test-case duration values:

                ...
                <passed />
                <tc-time>1.173351</tc-time></tc>
                <tp-time>5.399477</tp-time></tp>
                ...

        2. Use "xsltproc tests-resultss.xsl xxx.xml" to generate the html
           output.  This results in output similar to

                ...
                <td class="numeric"><tc-time>1.173351</tc-time>s</td>
                ...
                <td class="numeric"><tp-time>5.399477</tp-time>s</td>
                ...

        3. The bogus <tc-time>...</tc-time> and <tp-time>...</tp-time>
           tags prevent the style sheet from matching and applying the
           following attributes, and the output field ends up being
           left-justified:

                ...
                table.tcs-summary td.numeric {
                    text-align: right;
                    width: 1pt;
                }
                ...
        
>Fix:
        Sorry, my XML skills don't let me figure out the fix.  :9
        

>Unformatted: