I'm not sure if in the meantime you want to backout the change?
I'm not sure either.
We are trying to get a clean sheet on the Linux headful tests - although
there are a lot that are problem listed.
If we can't figure this out very soon, we'll minimally have to problem list
these tests
and have another bug open to address the cause .. well I suppose we need a bug
regardless
so I've filed https://bugs.openjdk.java.net/browse/JDK-8234107.
That can either be used for backout, follow on fix, or problem listing but
if we use it for backout or problem listing someone will need to file a new bug
to track the re-fix.
-phil.
On 11/13/19 11:23 AM, Mario Torre wrote:
Ok, I can reproduce the error now.
The test uses the VK_TAB to traverse the buttons, I admit this code is
quite... interesting.... I'll need to debug a bit more to see why the
focus logic for the wheel interest this. I'm not sure if in the
meantime you want to backout the change?
Cheers,
Mario
On Wed, Nov 13, 2019 at 7:28 PM Mario Torre
<neugens.limasoftw...@gmail.com> wrote:
Hi Phil,
This is strange, as I've run the full jtreg tests in addition to the
mouse event ones and didn't get any failure. Nonetheless, thanks for
spotting and apologise for breaking the build, I'll go through those
ASAP and see if there's a problem with the tests or something else, I
fail to see how this change would affect the modal dialog, since the
change only deal with mouse wheel event... but well...
Cheers,
Mario
Il giorno mer 13 nov 2019 alle ore 18:43 Phil Race
<philip.r...@oracle.com> ha scritto:
Mario,
Since you pushed this 4 modal dialog automated tests are failing - mostly -
on at least Ubuntu 16.04 and 18.04.
java/awt/Modal/ModalBlockingTests/BlockingWindowsSetModal6Test.java
java/awt/Modal/ModalBlockingTests/BlockingWindowsToolkitModal5Test.java
java/awt/Modal/ModalBlockingTests/BlockingWindowsDocModal2Test.java
java/awt/Modal/ModalBlockingTests/BlockingWindowsToolkitModal6Test.java
They fail in our nightly and I reproduced it locally,
They don't all always fail but never do they all pass : eg
jtreg -a -v
java/awt/Modal/ModalBlockingTests/BlockingWindowsSetModal6Test.java
java/awt/Modal/ModalBlockingTests/BlockingWindowsToolkitModal5Test.java
java/awt/Modal/ModalBlockingTests/BlockingWindowsDocModal2Test.java
java/awt/Modal/ModalBlockingTests/BlockingWindowsToolkitModal6Test.java
runner starting test:
java/awt/Modal/ModalBlockingTests/BlockingWindowsDocModal2Test.java
runner finished test:
java/awt/Modal/ModalBlockingTests/BlockingWindowsDocModal2Test.java
Failed. Execution failed: `main' threw exception:
java.lang.RuntimeException: DummyButton on Frame did not gain focus on
clicking. A document modal dialog and its parent are visible.: expected
true, was false
runner starting test:
java/awt/Modal/ModalBlockingTests/BlockingWindowsSetModal6Test.java
runner finished test:
java/awt/Modal/ModalBlockingTests/BlockingWindowsSetModal6Test.java
Failed. Execution failed: `main' threw exception:
java.lang.RuntimeException: DummyButton on Window did not gain focus on
clicking. The parent of the Window is null Dialog: expected true, was false
runner starting test:
java/awt/Modal/ModalBlockingTests/BlockingWindowsToolkitModal5Test.java
runner finished test:
java/awt/Modal/ModalBlockingTests/BlockingWindowsToolkitModal5Test.java
Failed. Execution failed: `main' threw exception:
java.lang.RuntimeException: DummyButton on Window did not gain focus on
clicking. The parent of the Window is TOOLKIT_MODAL Dialog: expected
true, was false
runner starting test:
java/awt/Modal/ModalBlockingTests/BlockingWindowsToolkitModal6Test.java
runner finished test:
java/awt/Modal/ModalBlockingTests/BlockingWindowsToolkitModal6Test.java
Passed. Execution successful
Test results: passed: 1; failed: 3
When I backout your change, the tests all pass again.
Prasanta also reported that backing out this change fixed them.
These tests aren't what I'd call 100% stable but there is definitely a
new problem here.
Can you check what you see on your end and we'll have to decide if
there's a follow on
fix or a back out whilst we think about it, or whether you believe there
was a latent
test problem ?
-phil.
On 11/12/19 8:58 AM, Mario Torre wrote:
On Mon, 2019-11-11 at 11:45 +0000, Dmitry Markov wrote:
Hi Mario,
The fix looks good to me.
Thanks, I pushed it.
Cheers,
Mario
--
pgp key: http://subkeys.pgp.net/ PGP Key ID: 80F240CF
Fingerprint: BA39 9666 94EC 8B73 27FA FC7C 4086 63E3 80F2 40CF
Java Champion - Blog: http://neugens.wordpress.com - Twitter: @neugens
Proud GNU Classpath developer: http://www.classpath.org/
OpenJDK: http://openjdk.java.net/projects/caciocavallo/
Please, support open standards:
http://endsoftpatents.org/
