RFR: 8241307: Marlin renderer is used by default in jdk8u

[Severin Gehwolf]

On Mon, 2020-03-23 at 12:29 +0300, Alexander Scherbatiy wrote:
> On 20.03.2020 19:07, Andrew Hughes wrote:
> 
> > On 20/03/2020 15:36, Hohensee, Paul wrote:
> > > For the fix to go into 8u252, label the issue with 'jdk8u-critical-request' and remove the 'jdk8u-fix-request' label. I removed 'openjdk8u' from the Affects Version/s list because it only affects openjdk8u252.
> > > 
> > > Paul
> > > 
> > > On 3/20/20, 3:17 AM, "jdk8u-dev on behalf of Laurent Bourgès" <jdk8u-dev-bounces at openjdk.java.net on behalf of bourges.laurent at gmail.com> wrote:
> > >      
> > >      Hi Andrew,
> > >      as 2 reviewers approved this patch, is it enough to push ?
> > >      
> > >      Should I add more labels to this jdk8u bug or wait for 8u maintainer
> > >      approval ?
> > >      
> > >      Could you then backport on OpenJDK8u 252 if you agree ?
> > >      
> > >      Cheers,
> > >      Laurent
> > >      
> > >      Le ven. 20 mars 2020 à 10:48, Alexander Scherbatiy <
> > >      alexander.scherbatiy at bell-sw.com> a écrit :
> > >      
> > >      > The fix looks good to me.
> > >      >
> > >      > Thanks,
> > >      >
> > >      > Alexander.
> > >      > On 20.03.2020 00:23, Laurent Bourgès wrote:
> > >      >
> > >      > Please review this jdk8u specific patch to restore the default behaviour
> > >      > (Marlin renderer is not the default RenderingEngine).
> > >      >
> > >      >
> > >      > Note: this impacts the coming JDK8 b252 release too, sorry.
> > >      >
> > >      >
> > >      >
> > >      > JBS: https://bugs.openjdk.java.net/browse/JDK-8241307
> > >      >
> > >      > webrev: http://cr.openjdk.java.net/~lbourges/marlin8u/marlin-8241307.0/
> > >      >
> > >      > Changes:
> > >      > - fixed the class order sun.java2d.pipe.RenderingEngine files to ensure
> > >      > Marlin is not at the last place (default service)
> > >      > - new test DefaultRenderingEngine that fails if Marlin is the default
> > >      > rendering engine
> > >      >
> > >      > Build: OK
> > >      > JTreg fails without patch:
> > >      > runner starting test: sun/java2d/marlin/DefaultRenderingEngine.java
> > >      > runner finished test: sun/java2d/marlin/DefaultRenderingEngine.java
> > >      > Failed. Execution failed: `main' threw exception:
> > >      > java.lang.RuntimeException: Marlin must not be the default RenderingEngine
> > >      > in jdk8u !
> > >      >
> > >      > JTreg passes with patch:
> > >      > runner starting test: sun/java2d/marlin/DefaultRenderingEngine.java
> > >      > runner finished test: sun/java2d/marlin/DefaultRenderingEngine.java
> > >      > Passed. Execution successful
> > >      >
> > >      > PS: Thanks to Alexander to have reported this problem today.
> > >      >
> > >      > Laurent
> > >      >
> > >      >
> > >      
> > > 
> > I think this is one for 8u262. We should have a cycle where Marlin is
> > available, not default, as with JFR.
> 
>   I believe that the issue affects 8u252 as well. If openjdk is built 
> from sources which do not include closed repository the default Ductus 
> rendering engine is not found so the latest registered one (currently 
> Marlin) is used as the default one.
> 
> To not make the Marlin as the default rendering engine in 8u252 the fix 
> puts Marlin before Pisces so Pisces rendering engine becomes the default 
> one as it was in 8u242.

That's right. We need this for 8u252:

./openjdk-8u252-b06/bin/java -showversion DefaultRenderingEngine
openjdk version "1.8.0_252-ea"
OpenJDK Runtime Environment (build 1.8.0_252-ea-b06)
OpenJDK 64-Bit Server VM (build 25.252-b06, mixed mode)

Exception in thread "main" java.lang.RuntimeException: Marlin must not be the default RenderingEngine in jdk8u !
	at DefaultRenderingEngine.main(DefaultRenderingEngine.java:16)

Andrew, please approve this for the 8u252 cycle. Once we flip the
default, we'll need to update the test.

Thanks,
Severin