<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"><head><meta http-equiv=Content-Type content="text/html; charset=us-ascii"><meta name=Generator content="Microsoft Word 15 (filtered medium)"><style><!--
/* Font Definitions */
@font-face
        {font-family:PMingLiU;
        panose-1:2 1 6 1 0 1 1 1 1 1;}
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:Kartika;
        panose-1:0 0 0 0 0 0 0 0 0 0;}
@font-face
        {font-family:"\@PMingLiU";
        panose-1:2 1 6 1 0 1 1 1 1 1;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:#0563C1;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:#954F72;
        text-decoration:underline;}
p.msonormal0, li.msonormal0, div.msonormal0
        {mso-style-name:msonormal;
        mso-margin-top-alt:auto;
        margin-right:0in;
        mso-margin-bottom-alt:auto;
        margin-left:0in;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
span.EmailStyle18
        {mso-style-type:personal-compose;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;
        font-family:"Calibri",sans-serif;}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]--></head><body lang=EN-US link="#0563C1" vlink="#954F72"><div class=WordSection1><p class=MsoNormal>Hi,<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Please review this webrev for JDK-8u backport.<o:p></o:p></p><p class=MsoNormal>Webrev: <a href="http://cr.openjdk.java.net/~ssreedharan/8180370/jdk8u-dev/webrev.00/">http://cr.openjdk.java.net/~ssreedharan/8180370/jdk8u-dev/webrev.00/</a><o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Main Bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8180370">https://bugs.openjdk.java.net/browse/JDK-8180370</a><o:p></o:p></p><p class=MsoNormal>JDK10 review thread: <a href="http://mail.openjdk.java.net/pipermail/awt-dev/2017-June/012881.html">http://mail.openjdk.java.net/pipermail/awt-dev/2017-June/012881.html</a><o:p></o:p></p><p class=MsoNormal><span lang=DE-CH>JDK10 changeset: </span><a href="http://hg.openjdk.java.net/jdk10/jdk10/jdk/rev/7d3e0c5b5e25"><span lang=DE-CH>http://hg.openjdk.java.net/jdk10/jdk10/jdk/rev/7d3e0c5b5e25</span></a><span lang=DE-CH><o:p></o:p></span></p><p class=MsoNormal><span lang=DE-CH><o:p> </o:p></span></p><p class=MsoNormal>The patch from JDK10 was not applied cleanly.<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Changes were made only in AWTView.m<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>The JDK10 patch was using the NSMutableString <i>useString</i> length to ensure that the fKeyEventsNeeded is reset only for a non-zero length string.<o:p></o:p></p><p class=MsoNormal>This NSMutableString <i>useString</i>  is however not available in JDK8u-dev.<o:p></o:p></p><p class=MsoNormal>This variable was added as part of fix for BUG JDK-8148555: [macosx] An uncaught exception was raised entering Emoji into JTextArea.<o:p></o:p></p><p class=MsoNormal>This however has not been backported to JDK8u-dev.<o:p></o:p></p><p class=MsoNormal>So replaced the use of NSMutableString <i>useString</i> length check with a check using the existing variables <i>utf16Length</i> and <i>utf8Length</i>.<o:p></o:p></p><p class=MsoNormal>This will not cause an issue even if JDK-8148555 is eventually backported.<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>I have tested out all the relevant Swing and AWT JTREG tests.<o:p></o:p></p><p class=MsoNormal>Also, ensured that there are no failures related to this on JPRT (both west and Stockholm) Queues.<o:p></o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal><o:p> </o:p></p><p class=MsoNormal>Regards,<o:p></o:p></p><p class=MsoNormal>Sreeprakash<o:p></o:p></p></div></body></html>