Closed Bug 130314 Opened 23 years ago Closed 23 years ago

comment close appears to be skipped

Categories

(Core :: DOM: HTML Parser, defect)

Product:
Core
Component:
DOM: HTML Parser
Platform:
x86
Windows XP
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
VERIFIED INVALID

People

(Reporter: David.Biesack, Assigned: harishd)

References

(
URL
)

Details

From Bugzilla Helper: User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:0.9.9+) Gecko/20020311 BuildID: 20020311 view the following html file. Some of the table rows are not rendered. If you remove the <!DOCTYPE ...> the table is rendered as expected, or if you remove the <!---------> comments. <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "DTD/xhtml1-transitional.dtd"> <html> <head> </head> <body bgcolor="#FFFFFF"> <table rules=all frame=border> <tr> <th>Option<th>Description <!--------------------------------------------------------------------------> <tr ALIGN="LEFT" VALIGN="TOP"> <td>-b directory<br> -basedir directory <td>Specify the starting directory <!--------------------------------------------------------------------------> <tr ALIGN="LEFT" VALIGN="TOP"> <td>-c<br> -clearjardir <td>Remove all bundles in the local jar directory before retrieving new files. <!--------------------------------------------------------------------------> <tr ALIGN="LEFT" VALIGN="TOP"> <td>-j local_jar_directory<br> -jar local_jar_directory <td>Specify the destination for the jar files. <!--------------------------------------------------------------------------> <tr ALIGN="LEFT" VALIGN="TOP"> <td>-p product_name<br> -product product_name <td>Specify the name of the product you are building. <!--------------------------------------------------------------------------> <tr ALIGN="LEFT" VALIGN="TOP"> <td>-r<br> -recursive <td>Recursively get the bundles. <!--------------------------------------------------------------------------> <tr ALIGN="LEFT" VALIGN="TOP"> <td>-s<br> -settings <td>Display current settings. <!--------------------------------------------------------------------------> <tr ALIGN="LEFT" VALIGN="TOP"> <td>-u<br> -usage <td>Display usage information. <!--------------------------------------------------------------------------> <tr ALIGN="LEFT" VALIGN="TOP"> <td>-v product_version<br> -version product_version <td>Specify the version of the product you are building. <!--------------------------------------------------------------------------> </table> </body> </html> Reproducible: Always Steps to Reproduce: 1. paste the html source into a file 2. view the html file with the browser Actual Results: You only see five table rows (including the header). The first <!--------------------------------------------------------------------------> appears to be parsed only as a comment open, the text until the next <!--------------------------------------------------------------------------> is therefore ignored as if it were the comment body. This only seems to happen with Expected Results: There should be 9 table rows (including the header row)
This should be valid since there are 74 dashes. In other words, there are 37 -- open-close matches, which is okay, even in XHTML AFAIK. See ongoing discussion of such issues in bug 130045.
Status: UNCONFIRMED → NEW
Ever confirmed: true
The HTML comment open sequence is "--". The HTML comment close sequence is "--". 74 dashes means 37 open/close items. Of these, there are 19 open and 18 close. So you get the rest of the doc commented out. resolving invalid. This is correct behavior.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → INVALID
Yep, my bad, I divided by two once, but should have divided again. Verifying.
Status: RESOLVED → VERIFIED
*** Bug 270256 has been marked as a duplicate of this bug. ***
You need to log in before you can comment on or make changes to this bug.