<AWT Dev> [9] Review Request: 8027324 The spec for Toolkit.setDynamicLayout() and Toolkit.isDynamicLayoutActive() needs to be clarified

Alexander Scherbatiy alexandr.scherbatiy at oracle.com
Thu Sep 24 12:59:11 UTC 2015

   The fix looks good to me.


On 9/23/2015 10:01 PM, Sergey Bylokhov wrote:
> Hello.
> Please review the fix for jdk9.
> We have 3 methods and one desktop property to support
> behavior during resize of the window:
> - Property "awt.dynamicLayoutSupported" describe support of this 
> feature by the current OS.
> - Method isDynamicLayoutActive() describe: is this feature currently 
> active or not.
> - isDynamicLayoutSet/setDynamicLayout is a possibility to change the 
> current state of the feature.
> The problem is that the method isDynamicLayoutActive() has no 
> description what happens if the OS supports(or nonsupports) the 
> feature, but the change of the state is not supported, which means 
> that the value, which was set in setDynamicLayout() is ignored. 
> Moreover this documentation states that isDynamicLayoutSet() is never 
> ignored.
> Documentation is updated using the same description as in 
> setDynamicLayout(); The existed test is updated to cover this situation.
> This bug causes a jck failure see JCK-7301760.
> ccc request will be created after the technical review.
> Bug: https://bugs.openjdk.java.net/browse/JDK-8027324
> Webrev can be found at: 
> http://cr.openjdk.java.net/~serb/8027324/webrev.00

More information about the awt-dev mailing list