On Thu, 22 Sep 2022 20:58:16 GMT, Harshitha Onkar <hon...@openjdk.org> wrote:

>> On Windows, the insets obtained for a Non-Resizable AWT Frame was different 
>> when frame.pack() was called and subsequent call to frame.getInsets() or 
>> frame.getPreferredSize(). Due to this, the actual and preferred size 
>> differed when frame.pack() was called for Non-Resizable frame (on Windows).  
>> 
>> Earlier the insets returned when frame.getInsets() was called, was that of a 
>> Resizable frame and not the correct insets associated with Non-Resizable 
>> frame. Fix is added to native code to get the correct insets. The test - 
>> AwtFramePackTest.java has been updated to test actual and expected/preferred 
>> size for both Resizable and Non-Resizable Frames.
>> 
>> The test is generic though the issue and fix is on Windows platform because 
>> the condition 
>> `frame.getSize() == frame.getPreferredSize()` should be true on all 
>> platforms when frame.pack() is called.
>> 
>> Following is the link to Windows System Metrics (used for native insets) - 
>> https://docs.microsoft.com/en-us/windows/win32/api/winuser/nf-winuser-getsystemmetrics
>
> Harshitha Onkar has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   removed unused extraBottomInsets, code cleanup

Marked as reviewed by aivanov (Reviewer).

-------------

PR: https://git.openjdk.org/jdk/pull/9954

Reply via email to