Hi Sergey, Thanks for your comment.
This new test only validates the new clipping algorithms ie it compares the rendering outputs with / without clipping enabled. As such algorithms are only available in Marlin 0.8.2 and the test uses new system properties to enable/disable clipping, I confirm it passes before (jdk9 or jdk10 before patch). To ensure it detects any regression, I manually inserted some bugs in the clipping code, and the test failed. Note: I should add another test @run to check the float variant too (and not only the double variant, the default in jdk10). Finally I could write a new performance test that would prove clipping is more efficient than before. Such test would fail before patch (timeout ?), but it is difficult to make it robust as it depends on the hw. Jim wrote a basic test in the jfx bug showing 300ms without but 2ms now => gain is high. A possible success condition would be: clipping gain > 10 or 50. Regards, Laurent Le 4 déc. 2017 11:11 PM, "Sergey Bylokhov" <[email protected]> a écrit : Hi, Laurent. On 29/11/2017 14:30, Laurent Bourgès wrote: > - added new ClipShapeTest (jtreg) that checks all possible combinations of > (cap / join) for random polyline (Stroker) and polygons (Filler) comparing > image outputs rendered with clipping enabled vs disabled > I have only one note that the test is passed before the fix, so if we will regress at some point later we will not catch this. -- Best regards, Sergey.
