From alexandre.iline at oracle.com Thu Oct 5 17:04:42 2017 From: alexandre.iline at oracle.com (Alexandre (Shura) Iline) Date: Thu, 5 Oct 2017 10:04:42 -0700 Subject: RFR 790204: NPE in org.netbeans.jemmy.Timeouts.getTimeout(String) Message-ID: Hi. Please take a look on this fix. The problem has been discovered in a repeated execution on tests. Fix is simple and there is a simple test which does show the problem with the fix reverted and works after the fix is applied. Bug: https://bugs.openjdk.java.net/browse/CODETOOLS-7902046 Webrev: http://cr.openjdk.java.net/~shurailine/7902046/webrev.00/ Thank you. Shura From alexandre.iline at oracle.com Mon Oct 9 13:58:24 2017 From: alexandre.iline at oracle.com (Alexandre (Shura) Iline) Date: Mon, 9 Oct 2017 06:58:24 -0700 Subject: RFR 790204: NPE in org.netbeans.jemmy.Timeouts.getTimeout(String) In-Reply-To: References: Message-ID: <6C1CB81B-8F32-48C6-96F9-7139C7EC0944@oracle.com> Thank you, Muneer. > On Oct 9, 2017, at 12:15 AM, Muneer Kolarkunnu wrote: > > Changes look good to me. > > Regards, > Muneer > > -----Original Message----- > From: Alexandre (Shura) Iline > Sent: Thursday, October 05, 2017 10:35 PM > To: jemmy-dev at openjdk.java.net > Cc: Muneer Kolarkunnu; Srinivas Mandalika > Subject: RFR 790204: NPE in org.netbeans.jemmy.Timeouts.getTimeout(String) > > Hi. > > Please take a look on this fix. The problem has been discovered in a repeated execution on tests. Fix is simple and there is a simple test which does show the problem with the fix reverted and works after the fix is applied. > > Bug: https://bugs.openjdk.java.net/browse/CODETOOLS-7902046 > Webrev: http://cr.openjdk.java.net/~shurailine/7902046/webrev.00/ > > Thank you. > > Shura > From alexey.fedorchenko at oracle.com Mon Oct 9 21:40:45 2017 From: alexey.fedorchenko at oracle.com (alexey.fedorchenko at oracle.com) Date: Mon, 09 Oct 2017 21:40:45 +0000 Subject: hg: code-tools/jemmy/v2: 7902046: NPE in org.netbeans.jemmy.Timeouts.getTimeout(String) Message-ID: <201710092140.v99LejL9006626@aojmv0008.oracle.com> Changeset: 35242e7b0424 Author: shurailine Date: 2017-10-09 14:12 -0700 URL: http://hg.openjdk.java.net/code-tools/jemmy/v2/rev/35242e7b0424 7902046: NPE in org.netbeans.jemmy.Timeouts.getTimeout(String) Reviewed-by: akolarkunnu + make/jemmy-v2-ii/.idea/.name + make/jemmy-v2-ii/.idea/compiler.xml + make/jemmy-v2-ii/.idea/encodings.xml + make/jemmy-v2-ii/.idea/misc.xml + make/jemmy-v2-ii/.idea/modules.xml ! src/org/netbeans/jemmy/Timeouts.java ! src/org/netbeans/jemmy/operators/FrameOperator.java ! test/org/netbeans/jemmy/scenario/MenuButtonTextTest.java + test/org/netbeans/jemmy/stress/TimeoutsGetSetTest.java From abdul.kolarkunnu at oracle.com Mon Oct 9 07:15:58 2017 From: abdul.kolarkunnu at oracle.com (Muneer Kolarkunnu) Date: Mon, 9 Oct 2017 00:15:58 -0700 (PDT) Subject: RFR 790204: NPE in org.netbeans.jemmy.Timeouts.getTimeout(String) In-Reply-To: References: Message-ID: Changes look good to me. Regards, Muneer -----Original Message----- From: Alexandre (Shura) Iline Sent: Thursday, October 05, 2017 10:35 PM To: jemmy-dev at openjdk.java.net Cc: Muneer Kolarkunnu; Srinivas Mandalika Subject: RFR 790204: NPE in org.netbeans.jemmy.Timeouts.getTimeout(String) Hi. Please take a look on this fix. The problem has been discovered in a repeated execution on tests. Fix is simple and there is a simple test which does show the problem with the fix reverted and works after the fix is applied. Bug: https://bugs.openjdk.java.net/browse/CODETOOLS-7902046 Webrev: http://cr.openjdk.java.net/~shurailine/7902046/webrev.00/ Thank you. Shura