[11u]: RFR(S): 8221823: Requested JDialog width is ignored

[Langer, Christoph]

Hi Goetz,

looks good now ��

Cheers
Christoph

> -----Original Message-----
> From: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>
> Sent: Freitag, 8. Mai 2020 12:03
> To: Langer, Christoph <christoph.langer at sap.com>; jdk-updates-dev <jdk-
> updates-dev at openjdk.java.net>
> Subject: RE: [11u]: RFR(S): 8221823: Requested JDialog width is ignored
> 
> Hi Christoph,
> 
> You are right, the lines are there. They are there twice,
> that's why I missed them.
> That's bad as the second line overwrites the first one ...
> so the mac exclude was basically disabled by the windows one.
> But this is fixed by this downport.
> 
> New webrev:
> http://cr.openjdk.java.net/~goetz/wr20/8221823-JDialog_width_ignored-
> jdk11/02/
> 
> Best regards,
>   Goetz.
> 
> 
> 
> > -----Original Message-----
> > From: Langer, Christoph <christoph.langer at sap.com>
> > Sent: Tuesday, April 28, 2020 9:31 AM
> > To: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>; jdk-updates-dev
> > <jdk-updates-dev at openjdk.java.net>
> > Subject: RE: [11u]: RFR(S): 8221823: Requested JDialog width is ignored
> >
> > Hi Goetz,
> >
> > I think the problem list entries exist in 11u-dev. I find them in lines 520 and
> > 521 of test/jdk/ProblemList.txt in my repo. Can you please check this
> again?
> >
> > Best regards
> > Christoph
> >
> > > -----Original Message-----
> > > From: jdk-updates-dev <jdk-updates-dev-bounces at openjdk.java.net>
> On
> > > Behalf Of Lindenmaier, Goetz
> > > Sent: Dienstag, 21. April 2020 15:52
> > > To: jdk-updates-dev <jdk-updates-dev at openjdk.java.net>
> > > Subject: [CAUTION] [11u]: RFR(S): 8221823: Requested JDialog width is
> > > ignored
> > >
> > > Hi,
> > >
> > > I please need review for this downport for parity with 11.0.8-oracle:
> > > http://cr.openjdk.java.net/~goetz/wr20/8221823-
> JDialog_width_ignored-
> > > jdk11/01/
> > >
> > > The problemlist patch does not apply.
> > > The tests are not problemlisted in 11u for windows. One of them is
> > > Problemlisted for mac, though. I left that line in place.
> > >
> > > Further I had to change a switch expression to a switch
> > > Statement in the test, else it would not compile.
> > >
> > > Original change:
> > > https://bugs.openjdk.java.net/browse/JDK-8221823
> > > https://hg.openjdk.java.net/jdk/jdk/rev/19adbbca4307
> > >
> > > Best regards,
> > >   Goetz.