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

[Artem Ananiev]

On 3/29/2012 7:11 PM, Anthony Petrov wrote:
> 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.

Thanks. This fix looks fine then.

Artem

> --
> 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