JDK-8187447: 2081 doc issues in jvmti.html

Sat Jan 13 01:05:19 UTC 2018

Updated the fix after offline discussion with Serguei:
http://cr.openjdk.java.net/~amenkov/8187447/webrev.01/

changes:
rollback replacement <p/> with <sep/> in jvmti.xml,
handle the case by transform in jvmti.xsl

Resulting jvmti.html is exactly the same.

--alex

On 01/11/2018 17:21, Alex Menkov wrote:
> Hi,
> 
> Please review a fix for doc issue.
> 
> jira: https://bugs.openjdk.java.net/browse/JDK-8187447
> webrev: http://cr.openjdk.java.net/~amenkov/8187447/webrev/
> 
> old jvmti.html: http://cr.openjdk.java.net/~amenkov/8187447/jvmti_0.html
> new jvmti.html: http://cr.openjdk.java.net/~amenkov/8187447/jvmti.html
> 
> specdiff reports no changes: 
> http://cr.openjdk.java.net/~amenkov/8187447/specdiff/diff.html
> 
> The fix:
> - converts jvmti.html from HTML4 to HTML5;
> - fixes all errors and almost all warnings reported by tidy and W3C HTML 
> Validator:
>    - empty <p>;
>    - unsupported attributes (align, border, width, cellpadding, 
> cellspacing, noshade, size, bgcolor, type);
>    - <dl> without nested <dt>;
>    - <a> with "name" attribute;
>    - table with column with no cells beginning in it (all cells have 
> colspan=2);
> - adds "lang" attribute for <html> (accessibility issue).
> 
> The only remaining warning (reported by W3C HTML Validator) is
>    Warning: Documents should not use about:legacy-compat, except if 
> generated by legacy systems that can't output the standard <!DOCTYPE 
> html> doctype.
> This is expected as the DOCTYPE legacy string 
> (http://w3c.github.io/html/single-page.html#doctype-legacy-string) was 
> added to HTML5 standard to handle issue with XLST which cannot produce 
> standard doctype.
> 
> --alex