Review request for 8169294:

Laurent Bourgès bourges.laurent at
Mon Nov 7 21:14:53 UTC 2016


2016-11-07 21:05 GMT+01:00 Jim Graham <james.graham at>:

> I'd like to see Kevin review the test as I'm not the best expert on our
> JUnit framework.

I just added @Test annotations and kept the jtreg tags in header (for
I could add asserts but JUnit does intercept any thrown exception and
reports a test failure in such case.

I managed to undestand how to run my test with gradle:
gradle :graphics:test  --tests

It is passing and the reports indicates that the 2 new tests are OK (report
in modules/

> It looks like it is mostly just going to emit some printouts about
> performance (using echo() and log()) and verify that we don't get any
> ArrayBounds related exceptions (or worse, OOME)?

Exactly, it is passing as there is no runtime exception and the performance
issue is fixed as indicated by the few logged lines.

> The only difference in I noted is that the Java2D version has
> an EXPAND_MIN which is 10, but you re-use INIT_SIZE, which is 20, here for
> the same purpose.

You're right; I think I didn't want to add an extra constant but if you
prefer being more consistent, I can prepare another webrev.


More information about the openjfx-dev mailing list