Re: [OpenJDK 2D-Dev] request for review: 7188093

2012-09-20 Thread Vadim Pakhnushev
Phil, I'm not sure about original test case, it uses cow.gif file. Moreover, it's not useful for this particular bug at all. I can open my version of the test which clearly shows the bug. I suppose I should post a webrev for this same CR with added test? Thanks, Vadim On 18.09.2012 22:32, Phi

Re: [OpenJDK 2D-Dev] request for review: 7188093

2012-09-20 Thread Phil Race
Ideally it should go under this bug id but its not the end of the world if we push it under another one. If you can provide it soon in jtreg form we can do it as part of this fix. Ideally it should be automated. -phil. On 9/20/2012 7:51 AM, Vadim Pakhnushev wrote: Phil, I'm not sure about or

Re: [OpenJDK 2D-Dev] request for review: 7188093

2012-09-20 Thread Vadim Pakhnushev
I copied original test .java and .html files, renamed and adjusted them for this case. Is it enough for jtreg? Automated test will require taking screenshot and comparing results, I'm not sure if this mechanism works good. If it is, I could develop automated test, Andrew mentioned that there is

Re: [OpenJDK 2D-Dev] request for review: 7188093

2012-09-20 Thread Clemens Eisserer
Hi, > Automated test will require taking screenshot and comparing results, I'm not > sure if this mechanism works good. It should be possible to render to a VolatileImage, which allows direct readback without taking a screenshot. - Clemens

Re: [OpenJDK 2D-Dev] request for review: 7188093

2012-09-20 Thread Vadim Pakhnushev
That's good, I will try to improve my test to be automated then. Thanks, Vadim On 21.09.2012 0:40, Clemens Eisserer wrote: Hi, Automated test will require taking screenshot and comparing results, I'm not sure if this mechanism works good. It should be possible to render to a VolatileImage, w