<AWT Dev> [8] Review request for 7124428: [macosx] Frame.setExtendedState() doesn't work for undecorated windows

Anthony Petrov anthony.petrov at oracle.com
Thu Mar 29 08:11:50 PDT 2012


Hi Artem,

On 03/29/12 16:34, Artem Ananiev wrote:
>>> 2. Will this fix work in the case when an undecorated window goes
>>> fullscreen and then changes its bounds while being in the fullscreen
>>> mode?
>>
>> The full screen mode isn't an extended state. Also, my fix doesn't
>> affect the setBounds() implementation, so I don't see how this issue may
>> be relevant to this fix. Please elaborate.
>
> window.setBounds(x, y, width, height)
> graphicsDevice.setFullscreenWindow(window)
> window.setBounds(newX, newY, newWidth, newHeight)
> graphicsDevice.setFullscreenWindow(null)
>
> What will be the window bounds in this case?

In a private conversation we figured we wanted to replace the 
setFullScreenWindow() with a setExtendedState(MAXIMIZED_BOTH) in the 
above scenario, in which case it makes sense in the context of the fix.

However, it is clear that this issue affects other platforms as well. 
Note that currently the behavior of setBounds() in the MAXIMIZED_BOTH 
(or ICONIC) state is unspecified.

I've filed 7157742 to evaluate this issue separately from this fix as we 
might want to introduce more changes to other platforms code, and 
clarify the Java specification for this case, too.

--
best regards,
Anthony

>
> Thanks,
>
> Artem
>
>> --
>> best regards,
>> Anthony
>>
>>>
>>> Thanks,
>>>
>>> Artem
>>>
>>> On 3/23/2012 7:49 PM, Anthony Petrov wrote:
>>>> Hello,
>>>>
>>>> Please review a fix for
>>>> http://bugs.sun.com/view_bug.do?bug_id=7124428 at:
>>>>
>>>> http://cr.openjdk.java.net/~anthony/8-12-setExtendedStateUndecorated-7124428.0/
>>>>
>>>>
>>>>
>>>>
>>>> This is a direct forward-port of the same fix from 7u4.
>>>>
>>>> More details:
>>>> http://mail.openjdk.java.net/pipermail/macosx-port-dev/2012-March/003630.html
>>>>
>>>>
>>>>
>>>>
>>>> --
>>>> best regards,
>>>> Anthony



More information about the awt-dev mailing list