Hi, Please find the updated Webrev at:
http://cr.openjdk.java.net/~sveerabhadra/8190192/webrev.01/ The issue was that upon clicking on the title bar twice would invoke windowShouldZoom() which was returning false always to the appkit, thereby blocking the window from getting zoomed all the time. Now the logic is simplified as there is an existing zoom state of the window being maintained internally by the appkit and my change would use it instead of trying to make up our own logic. Thanks and regards, Shashi From: Philip Race Sent: Sunday, October 29, 2017 10:16 PM To: shashidhara veerabhadraiah <shashidhara.veerabhadra...@oracle.com> Cc: Sergey Bylokhov <sergey.bylok...@oracle.com>; awt-dev@openjdk.java.net Subject: Re: <AWT Dev> [10] JDK-8190192: Double click on the title bar no longer repositions the window Hello, > This seems a regression in JDK 8 as it worked fine in 7u80. That is too vague for my taste. Precisely what fix caused this regression ? Once you find that you may find some additional insight into the issue .. and not regress something else. >From a quick scan of the history of AWTWindow.m my top candidate is 8026143: [macosx] Maximized state could be inconsistent between peer and frame So if this regression starts from 8b115 that is quite likely the cause. Also why is it not possible to include an automated Robot regression test ? -phil. On 10/29/17, 7:51 AM, shashidhara veerabhadraiah wrote: Hi All, Please review a fix for the below bug: Bug: https://bugs.openjdk.java.net/browse/JDK-8190192 Webrev: HYPERLINK "http://cr.openjdk.java.net/%7Esveerabhadra/8190192/webrev.00/"http://cr.openjdk.java.net/~sveerabhadra/8190192/webrev.00/ Summary: Double clicking on the title bar of a java frame supposed to maximise the window which is not happening. Above fix makes corrects that behaviour by listening to double clicks on the title bar and accordingly raising a toggle full screen functionality on the window. Thanks and regards, Shashi
