<AWT Dev> [10] Review request for JDK-8180370: Characters are skipped on input of Korean text on OS X

[Sreeprakash Sreedharan]

Thanks for the inputs Phil.

I have updated the code and the comments.

Updated Webrev : http://cr.openjdk.java.net/~rpatil/8180370/webrev.01/

The issue was that any non-alpha character(numbers and symbols) typed immediately after a Korean character was getting skipped.
The non-alpha character will come if you type again.
For example, an exclamation mark (!) entered after the Korean character for q will not show up.
However, if you type it again it will appear. But instead of having 2 exclamation marks we have just one.

I had tested it out with 2-Set Korean, Wubi Xing (Chinese) and ABC - Extended keyboard layouts on mac with different input methods .

I had run all the manual and automatic regression tests for all text based controls in awt (like TextField, TextArea etc) and swing (like JTextField, JTextArea, JTextPane, JEditorPane etc) and did not find any issues.

Thanks,
Sreeprakash


From: Phil Race 
Sent: Friday, June 2, 2017 9:44 PM
To: Sreeprakash Sreedharan <sreeprakash.s at oracle.com>; awt-dev at openjdk.java.net
Subject: Re: <AWT Dev> [10] Review request for JDK-8180370: Characters are skipped on input of Korean text on OS X

I am not familiar with this code but I have a few comments anyway

1. I dislike cluttering the source with bug ids. If we did that for every fix
    quite soon the source code would be a mess of semi-random numbers.
    Anyone who really wants to know when this change was made has the history

2. if is not a function. So "if(" -> "if ("

3. When a marked text -> When marked text

4. "!" is not a "special character" .. its quite ordinary .. so what do you mean ?

5. What testing have you done to make sure no other cases are broken by this change ?
    The new test is manual and I'd bet that most tests that might cover this are manual
    I'd expect to hear that you have tested different input scenarios such as a couple
    of input methods/locales, and AWT and Swing input with a representative set of
    input as well as running all the relevant regression tests.

-phil.
On 06/02/2017 07:48 AM, Sreeprakash Sreedharan wrote:
Hi All,

Kindly review the fix for JDK10.
Bug: https://bugs.openjdk.java.net/browse/JDK-8180370
Webrev: http://cr.openjdk.java.net/~rpatil/8180370/webrev.00/
Issue: Special characters (like !,/\<> ) were getting skipped when immediately entered after a marked text on MacOSX.
Fix: Made sure that fKeyEventsNeeded flag is reset, when a non-marked text is encountered, so that it doesn't get ignored by key down.

Note: Since the keyboard layout has to be changed to Korean, I have added a manual test case wherein the user is prompted to change to Korean keyboard layout and then execute the test.


Regards,
Sreeprakash