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.

Reply via email to