Could you review the updated fix:
http://cr.openjdk.java.net/~alexsch/7092283/webrev.01
- the locationByPlatform property is updated by setter method
- the open and closed tests are passed with the fix except one which
fails without the fix too:
java/awt/Dialog/MakeWindowAlwaysOnTop/MakeWindowAlwaysOnTop.java
Thanks,
Alexandr.
On 9/25/2013 5:37 PM, Anthony Petrov wrote:
Hi Alexander,
The setter and getter for this property access the boolean flag under
the tree lock. I suppose this was the locking design for this field.
So you should reset it under this lock, too. You could also go ahead
and add proper locking to other methods that access it (e.g. show(),
etc.)
Please run Window, Frame, and Dialog automatic regression tests (open
and closed) to ensure no regressions after this change.
--
best regards,
Anthony
On 09/25/2013 04:50 PM, Alexander Scherbatiy wrote:
Hello,
Could you review the fix:
bug: https://bugs.openjdk.java.net/browse/JDK-7092283
webrev: http://cr.openjdk.java.net/~alexsch/7092283/webrev.00
The Window.hide() method does not clear the locationByPlatform
property.
Thanks,
Alexandr.
