<AWT Dev>  Review request for 8014212: Robot captures black screen
Sergey.Bylokhov at oracle.com
Mon Aug 3 16:05:59 UTC 2015
On 28.07.15 15:49, Alexander Zvegintsev wrote:
> Hello Sergey,
> please see the updated version
> GetPrimitiveArrayCritical() is used instead of manual array allocation.
> On 07/27/2015 08:35 PM, Alexander Zvegintsev wrote:
>> Please see the updated version:
>> robot makes shot of intersection of provided rectangle with root
>> window. Old robot implementation works for negative x and y,
>> but may return garbage outside of root window behavior, so it was
>> updated too.
>> ShapeNotSetSometimes reduced iteration count to prevent killing by
>> jtreg for timeout.
>> TranslucentJAppletTest updated to use getPixelColor() directly.
>> On 07/24/2015 07:08 AM, Sergey Bylokhov wrote:
>>> Hi, Alexander.
>>> I am not sure that an assumption that we should not make a screen
>>> shot if the x and y are negative is correct. I guess that we should
>>> take a screenshot of the intersection of the requested area and the
>>> Can you also check this comment in the TranslucentJAppletTest.java:
>>> 106 // unfortunately, robot.getPixelColor() doesn't work
>>> for some unknown reason
>>> 107 // Color newColor2 = r.getPixelColor(200, 200);
>>> On 23.07.15 19:37, Alexander Zvegintsev wrote:
>>>> please review the fix
>>>> for the issue
>>>> This fix trying to use gdk_pixbuf_get_from_drawable ()  to
>>>> capture screenshot(basic image data handling could be found here).
>>>> If it fails then we come back to our old method.
>>>> Translucency support is also covered by this fix, but there is
>>>> another issue with toFront() call which blocks passing of a JCK test.
>>>> P.S. On Solaris 11 windows appears with animation, thus sleep()
>>>> call was added to work around this.
>>>>  https://bugs.openjdk.java.net/browse/JDK-7043845 Robot should
>>>> be able take screenshots with translucent windows
>>> Best regards, Sergey.
Best regards, Sergey.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the awt-dev