Hi,

 

Please review this webrev for JDK-8u backport.

Webrev: http://cr.openjdk.java.net/~ssreedharan/8180370/jdk8u-dev/webrev.00/

 

Main Bug: https://bugs.openjdk.java.net/browse/JDK-8180370

JDK10 review thread: 
http://mail.openjdk.java.net/pipermail/awt-dev/2017-June/012881.html

JDK10 changeset: http://hg.openjdk.java.net/jdk10/jdk10/jdk/rev/7d3e0c5b5e25

 

The patch from JDK10 was not applied cleanly.

 

Changes were made only in AWTView.m

 

The JDK10 patch was using the NSMutableString useString length to ensure that 
the fKeyEventsNeeded is reset only for a non-zero length string.

This NSMutableString useString  is however not available in JDK8u-dev.

This variable was added as part of fix for BUG JDK-8148555: [macosx] An 
uncaught exception was raised entering Emoji into JTextArea.

This however has not been backported to JDK8u-dev.

So replaced the use of NSMutableString useString length check with a check 
using the existing variables utf16Length and utf8Length.

This will not cause an issue even if JDK-8148555 is eventually backported.

 

I have tested out all the relevant Swing and AWT JTREG tests.

Also, ensured that there are no failures related to this on JPRT (both west and 
Stockholm) Queues.

 

 

Regards,

Sreeprakash

Reply via email to