Thanks, that looks much better to me.
/Erik
On 2020-04-16 10:02, Igor Ignatyev wrote:
Hi Erik,
sure, I've actually replaced one long statement w/ multiple shorter
ones, which made the comment section redundant --
http://cr.openjdk.java.net/~iignatyev//8240904/webrev.01
Thanks,
-- Igor
On Apr 16, 2020, at 6:17 AM, Erik Joelsson <erik.joels...@oracle.com
<mailto:erik.joels...@oracle.com>> wrote:
Looks ok to me. Would it be possible to break up the long lines a bit
to improve readability? Backslash escape for newlines should work in
properties files.
/Erik
On 2020-04-15 22:22, Igor Ignatyev wrote:
http://cr.openjdk.java.net/~iignatyev//8240904/webrev.00
35 lines changed: 26 ins; 0 del; 10 mod
Hi all,
8233827[1] which added screenshots to so-called failure handler had
an unexpected side-effect on linux, where users might observer
flashes each time a screenshot is taken, which, to put it mildly, is
annoying. the patch replaces gnome-screenshot app w/ calling
java.awt API to make a screenshot. the patch also uses the same
solution to make screenshots on windows and solaris (which
previously didn't save screenshots on failures). we still use native
app on mac as using java.awt API requires accessibility permissions,
which might be as annoying as flashes on linux.
[1] https://bugs.openjdk.java.net/browse/JDK-8233827
JBS: https://bugs.openjdk.java.net/browse/JDK-8240904
webrev: http://cr.openjdk.java.net/~iignatyev//8240904/webrev.00
testing: verified that screenshots are successfully generated on
headful systems, and headless systems ignore errors
('java.awt.AWTException: headless environment' from j.a.Robot::<init>).
Thanks,
-- Igor