<AWT Dev> RFR: 8212676 AIX's CDE/MWM support

[Ichiroh Takiguchi]

Hello again.

Could you give me review comment and/or suggestion ?

Bug:    https://bugs.openjdk.java.net/browse/JDK-8212676
Change: https://cr.openjdk.java.net/~itakiguchi/8212676/webrev.00/

Thanks,
Ichiroh Takiguchi

On 2018-11-26 21:27, Ichiroh Takiguchi wrote:
> Hello.
> 
> Could you review the fix ?
> 
> Bug:    https://bugs.openjdk.java.net/browse/JDK-8212676
> Change: https://cr.openjdk.java.net/~itakiguchi/8212676/webrev.00/
> 
> Test instructions and screen shots are in JDK-8212676.
> 
> GUI environment for AIX platform still needs CDE support.
> This fix is required to avoid unexpected working behavior on AIX 
> platform.
> 
> I'd like to obtain a sponsor for this issue.
> 
> Thanks,
> Ichiroh Takiguchi
> IBM Japan, Ltd.
> 
> On 2018-06-18 18:57, Ichiroh Takiguchi wrote:
>> Hello.
>> 
>> This fix is really required for AIX's GUI.
>> System color setting and window manager's working behavior are very 
>> important.
>> 
>> I appreciate any feedback, and how I would go about obtaining a
>> sponsor and contributor ?
>> 
>> On 2018-05-25 11:02, Ichiroh Takiguchi wrote:
>>> Hello Phil.
>>> 
>>> webrev file was extracted.
>>> Please see
>>>   http://cr.openjdk.java.net/~aleonard/AIX_GUI/webrev.00/
>>> 
>>> On 2018-05-19 02:48, Ichiroh Takiguchi wrote:
>>>> Hello Phil.
>>>> 
>>>> Webrev.zip file is stored into
>>>>   http://cr.openjdk.java.net/~aleonard/AIX_GUI/webrev-aixgui.zip
>>>> 
>>>> Test programs are also stored:
>>>> No testcase is available for FontUtilities.java and 
>>>> XDecoratedPeer.java.
>>>> 
>>>> MotifColorUtilities.java
>>>>   http://cr.openjdk.java.net/~aleonard/AIX_GUI/SystemColorTest2.java
>>>> Run SystemColorTest2, system colors should be displayed
>>>> AIX sample is
>>>>   http://cr.openjdk.java.net/~aleonard/AIX_GUI/aix_systemcolor.txt
>>>> 
>>>> XWM.java
>>>>   http://cr.openjdk.java.net/~aleonard/AIX_GUI/XWMTest1.java
>>>> On AIX CDE, isMotif and isCDE were true.
>>>> On AIX MWM, every entry is false.
>>>> 
>>>> XWindowPeer.java
>>>>   http://cr.openjdk.java.net/~aleonard/AIX_GUI/JFrameTest.java
>>>> On AIX CDE, click inside of "Non-Focusable" window (not window 
>>>> frame).
>>>> Window focus should not be changed because of "click on focus" 
>>>> feature.
>>>> But input focus is moved to "Non-Focusable" window.
>>>> 
>>>> 
>>>> On 2018-05-18 01:00, Phil Race wrote:
>>>>> I think we'd need to see the actual proposed changes and understand
>>>>> the implications
>>>>> for ongoing support as we no longer support any platform which has 
>>>>> a
>>>>> CDE desktop.
>>>>> Solaris 11.3 uses Gnome, so we'd be more inclined to be ripping out
>>>>> such support rather
>>>>> than adding to it.
>>>>> 
>>>>> -phil.
>>>>> 
>>>>> On 05/17/2018 04:18 AM, Ichiroh Takiguchi wrote:
>>>>>> Hello,
>>>>>> IBM would like to contribute AIX's CDE (Common Desktop 
>>>>>> Environment) DTWM (Desktop Window Manager) /MWM (Motif Window 
>>>>>> Manager) support to OpenJDK project.
>>>>>> 
>>>>>> I'd like contribute following 5 files:
>>>>>> 
>>>>>> M src/java.desktop/share/classes/sun/font/FontUtilities.java
>>>>>> (Add isAIX flag to determine AIX platform for GUI environment)
>>>>>> M 
>>>>>> src/java.desktop/unix/classes/sun/awt/X11/MotifColorUtilities.java
>>>>>> (Add High Color support on CDE, OpenJDK just supports Medium 
>>>>>> Color) [1]
>>>>>> M src/java.desktop/unix/classes/sun/awt/X11/XDecoratedPeer.java
>>>>>> (Avoid miss calculation for window position under DTWM/MWM by 
>>>>>> XMapRaised/XMapWindow)
>>>>>> M src/java.desktop/unix/classes/sun/awt/X11/XWM.java
>>>>>> (Detect MWM on AIX platform)
>>>>>> M src/java.desktop/unix/classes/sun/awt/X11/XWindowPeer.java
>>>>>> (Add non-focusable window support on DTWM/MWM for AIX, because 
>>>>>> DTWM/MWM does not have enough features for ICCCM)
>>>>>> 
>>>>>> I appreciate any feedback please, and how I would go about 
>>>>>> obtaining a sponsor and contributor ?
>>>>>> 
>>>>>> Thanks,
>>>>>> Ichiroh Takiguchi
>>>>>> IBM Japan, Ltd.
>>>>>> 
>>>>>> [1] 
>>>>>> https://docs.oracle.com/cd/E19253-01/806-7492/fontsandcolors-15233/index.html
>>>>>>