Re: [9] Review request for 8162840: Desktop. enableSuddenTermination() has no effect

2016-08-22 Thread Sergey Bylokhov
Ok, then the current fix looks fine. On 17.08.16 10:22, Yuri Nesterenko wrote: In fact, we have a bunch of them in the func. workspace, all more or less easily portable to jtreg. I'll move them in a bulk, time permitting. -yan On 08/16/2016 07:47 PM, Alexander Zvegintsev wrote: Sure, but it

Re: [9] Review request for 8162840: Desktop. enableSuddenTermination() has no effect

2016-08-17 Thread Yuri Nesterenko
In fact, we have a bunch of them in the func. workspace, all more or less easily portable to jtreg. I'll move them in a bulk, time permitting. -yan On 08/16/2016 07:47 PM, Alexander Zvegintsev wrote: Sure, but it is already created by Yuri and attached to the issue. I can include it with this

Re: [9] Review request for 8162840: Desktop. enableSuddenTermination() has no effect

2016-08-16 Thread Alexander Zvegintsev
Sure, but it is already created by Yuri and attached to the issue. I can include it with this fix, however as far I know SQE has a separate task to cover this JEP with tests and this test is a part of this task. On 8/16/16 7:27 PM, Sergey Bylokhov wrote: Hi, Alexander. Is it possible to crea

Re: [9] Review request for 8162840: Desktop. enableSuddenTermination() has no effect

2016-08-16 Thread Sergey Bylokhov
Hi, Alexander. Is it possible to create a test for the fix? On 16.08.16 19:20, Alexander Zvegintsev wrote: Hello, please review the fix http://cr.openjdk.java.net/~azvegint/jdk/9/8162840/00/ for the issue https://bugs.openjdk.java.net/browse/JDK-8162840 We don't call QuitHandler and using

[9] Review request for 8162840: Desktop. enableSuddenTermination() has no effect

2016-08-16 Thread Alexander Zvegintsev
Hello, please review the fix http://cr.openjdk.java.net/~azvegint/jdk/9/8162840/00/ for the issue https://bugs.openjdk.java.net/browse/JDK-8162840 We don't call QuitHandler and using default QuitStrategy if sudden termination is enabled. -- Thanks, Alexander.