Hi Sergey, thanks for looking at the change.
Unfortunately there popped up four more test requiring a 'head' in the repo tonight. I added them, too: http://cr.openjdk.java.net/~goetz/wr17/8191678-headfulKey/webrev.02/ Is it still fine? Thanks, Goetz. > -----Original Message----- > From: Sergey Bylokhov [mailto:[email protected]] > Sent: Mittwoch, 29. November 2017 14:49 > To: Lindenmaier, Goetz <[email protected]>; awt- > [email protected] > Subject: Re: <AWT Dev> RFR(S): 8191678: [TESTBUG] Add keyword headful in > java/awt and javax tests. > > Looks fine. > > On 29/11/2017 00:01, Lindenmaier, Goetz wrote: > > Hi, > > > > I added @key headful to some more tests where it was missing. > > Also, I fixed some NPEs I get when running the tests headless. > > Please review. > > http://cr.openjdk.java.net/~goetz/wr17/8191678-headfulKey/webrev.01/ > > > > Best regards, > > Goetz. > > > > > -- > Best regards, Sergey.
