<AWT Dev> Subject: <AWT dev>[11] Review request for JDK-8208125: Cannot input text into JOptionPane Text Input Dialog

Prasanta Sadhukhan prasanta.sadhukhan at oracle.com
Thu Aug 9 05:47:23 UTC 2018


looks fine.

Regards
Prasanta
On 8/9/2018 2:51 AM, Philip Race wrote:
> +1
>
> -phil.
>
> On 8/8/18, 5:56 AM, Manajit Halder wrote:
>>
>> Hi Phil,
>>
>> Thanks for your review comment.
>>
>> Reverted the code optimization and test code as per the suggestion. 
>> Now the code change is same as the changeset e0b025915be8.
>> Verified the code reversion with all automated AWT JTreg tests, JCK 
>> automated tests, reported JCK manual tests and SwingSet2 test cases.
>>
>> The test (UnfocusableMaximizedFrameResizablity.java) passes on 
>> Windows and Ubuntu and fails on Mac, which is expected.
>>
>> Issue created to fix the original issue: 
>> https://bugs.openjdk.java.net/browse/JDK-8209123
>>
>>
>> Please review the changes:
>> http://cr.openjdk.java.net/~mhalder/8208125/webrev.01/ 
>> <http://cr.openjdk.java.net/%7Emhalder/8208125/webrev.01/>
>>
>> Regards,
>> Manajit
>>
>>
>> On 04/08/18 2:42 AM, Phil Race wrote:
>>> So you are saying the fix here cumulatively backs out the fix for 
>>> 7158623 and 8204860
>>> and reverts the product code to what it was before and re-introduces
>>> 7158623 which affected MacOS on all releases of JDK (7 and onwards) ?
>>> I expect that is OK. We can live with a 6 year old bug in 11 as much 
>>> better
>>> than a stop-ship TCK failure.
>>> However I'd prefer that you "exactly" reverted the fix
>>>
>>> Originally we had
>>> - final boolean resizable = isFrame ? ((Frame)target).isResizable() 
>>> : (isDialog ? ((Dialog)target).isResizable() : false); you are 
>>> proposing to keep the rewrite of that condition in the new fix. It 
>>> is much easier for me to be sure of the equivalence if you revert to 
>>> byte-for-byte what it was before we started. Ironically, the only 
>>> other change to this file since you started on this was also backed 
>>> out ! So once you are done we should be back to the same content as 
>>> changeset e0b025915be8. That changeset was also touching maximise so 
>>> I definitely think we want to make sure JCK passes.
>>>
>>> Have you run the JCK tests on Mac to make sure there are no surprises ?
>>> We don't have a lot of time left here.
>>>
>>> Please enumerate all tests you have run on this "reversion" fix.
>>> I expect it to include all automated jtreg + JCK tests, related 
>>> manual tests, + SwingSet2
>>> as well as the specific regression test.
>>>
>>> Its a bit hard to tell what you are doing with the test, its not an 
>>> exact reversion
>>> You deleted a lot of machinery and I suppose you want to keep that 
>>> part, just revert
>>> the parts that verify the functionality you've backed out.
>>> But you should make sure that if I as I expect the reverted test now 
>>> fails on Mac,
>>> and since it is automated, this fix should include problem listing 
>>> it on Mac.
>>> If it does not fail on Mac .. I'd like to know why .. since this was 
>>> the test that originally failed.
>>> Also you should make sure it definitely still passes on Windows + Linux.
>>>
>>> And can you file that new bug, with the full back story, and 
>>> reference it here ?
>>>
>>> -phil.
>>>
>>> On 08/03/2018 04:23 AM, Manajit Halder wrote:
>>>> Hi All,
>>>>
>>>> Please review the fix for JDK11.
>>>>
>>>> 

Bug:
>>>> 
https://bugs.openjdk.java.net/browse/JDK-8208125 
>>>> <https://bugs.openjdk.java.net/browse/JDK-8208125>
>>>>
>>>> 
Webrev: 

>>>> http://cr.openjdk.java.net/~mhalder/8208125/webrev.00/ 
>>>> <http://cr.openjdk.java.net/%7Emhalder/8208125/webrev.00/>
>>>>
>>>> 
Issue:
>>>>     This issue and JDK-8208127 was caused due to regression caused 
>>>> by fix for the issue JDK-8204860.
>>>>
>>>> Fix:
>>>>     I had a relook at the original issue fixed on Windows OS 
>>>> (JDK-4980161 <https://bugs.openjdk.java.net/browse/JDK-4980161>) 
>>>> and found that the fixes proposed by my earlier fixes were wrong 
>>>> w.r.t to the behaviour on Windows and Ubuntu. The issue 
>>>> (JDK-4980161) on Windows was that “non-focusable maximized frame 
>>>> was resizable” but a focusable maximized frame was already 
>>>> non-resizable (working as expected).
>>>>
>>>> Please read the problem description of issue JDK-4980161: which 
>>>> says “Typically a maximized frame is not supposed to be resizable.”
>>>>
>>>> I verified the behaviour on Windows and Ubuntu and found that a 
>>>> maximized Frame (frame.setExtendedState(Frame.MAXIMIZED_BOTH)) is 
>>>> not resizable irrespective of Frame’s focusable behaviour. On Mac 
>>>> OS this behaviour is not implemented i.e. the problem is “maximized 
>>>> frame is resizable”.
>>>>
>>>> So the problem which needs to fixed on Mac OS is “Maximized frame 
>>>> should not be resizable irrespective of its focusability”
>>>>
>>>> Therefor I have reverted my changes to the original state (before 
>>>> fix of issue JDK-7158623 
>>>> <https://bugs.openjdk.java.net/browse/JDK-7158623>) and propose to 
>>>> fix the problem on Mac by creating a new issue.
>>>>
>>>> 
Regards,
>>>> 
Manajit
>>>>
>>>>
>>>
>>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20180809/e7e0b5de/attachment-0001.html>


More information about the awt-dev mailing list