Hi Artem , I'll have a try about the testcase.
On Mon, Jan 16, 2012 at 11:56 PM, Artem Ananiev <[email protected]>wrote: > Hi, Sean, > > text components in XToolkit use Swing peers, so this change should better > be reviewed on the swing-dev alias. To save everybody's time, I've asked > Alex and Pavel from the Swing team (in CC) to take a look and they confirm > the fix looks fine. > > Did you consider creating a new regression test for this fix? I'm not sure > it's easy to detect if caret is visible in the text component or not, > though. > > Thanks, > > Artem > > > On 1/16/2012 7:08 AM, Sean Chou wrote: > >> Hi all, >> >> I made a patch for bug 7129742, >> http://bugs.sun.com/**bugdatabase/view_bug.do?bug_**id=7129742<http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7129742>. >> The webrev link is : http://cr.openjdk.java.net/~** >> zhouyx/7129742/webrev.00/<http://cr.openjdk.java.net/~zhouyx/7129742/webrev.00/> >> >> The solution is very simple, just set the cursor visible. >> >> Please have a look and give some comments. Thanks. >> >> -- >> Best Regards, >> Sean Chou >> >> -- Best Regards, Sean Chou
