Thanks a lot, Alex!
Serguei

On 7/1/19 11:25, Alex Menkov wrote:
+1

--alex

On 06/28/2019 21:44, David Holmes wrote:
Hi Serguei,

Looks good and trivial.

Thanks,
David

On 28/06/2019 9:55 pm, serguei.spit...@oracle.com wrote:
Please, review a fix for:
   https://bugs.openjdk.java.net/browse/JDK-8226603

Webrev:
http://cr.openjdk.java.net/~sspitsyn/webrevs/2019/8226603-jvmti-a11y.1/


Summary:
   The change fixes the a11y issues reported by the Axe tool.


Testing:
   Alex checked that the Axe tool reports noting with the updated jvmti.html.    Also, the DocCheck reported the following heading errors for the original jvmti.html:

% grep heading report0/accessibility.log
0/jvmti.html:30: headings omitted
0/jvmti.html:27: Note: This is the previous heading
0/jvmti.html:880: headings omitted
0/jvmti.html:879: Note: This is the previous heading
0/jvmti.html:5308: headings omitted
0/jvmti.html:5253: Note: This is the previous heading
0/jvmti.html:7700: headings omitted
0/jvmti.html:7607: Note: This is the previous heading
0/jvmti.html:19857: headings omitted
0/jvmti.html:19797: Note: This is the previous heading
0/jvmti.html:23821: headings omitted
0/jvmti.html:23593: Note: This is the previous heading
0/jvmti.html:24546: headings omitted
0/jvmti.html:24416: Note: This is the previous heading
0/jvmti.html:25404: headings omitted
0/jvmti.html:25362: Note: This is the previous heading
0/jvmti.html:30724: headings omitted
0/jvmti.html:30723: Note: This is the previous heading

Now all these errors are gone.

Thanks,
Serguei


Reply via email to