[8u] RFR: 8159690: [TESTBUG] Mark headful tests with @key headful.

Hohensee, Paul hohensee at amazon.com
Thu Oct 29 18:48:01 UTC 2020


As long as the change to IncorrectUnmanagedImageSourceOffset.java gets in somewhere, I'm fine with it.

Thanks,
Paul

On 10/29/20, 10:30 AM, "Zhengyu Gu" <zgu at redhat.com> wrote:

    Hi Paul,

    >   /*
    > - * Copyright (c) 2014, Oracle and/or its affiliates. All rights reserved.
    > + * Copyright (c) 2014, 2016, Oracle and/or its affiliates. All rights reserved.
    >    * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
    >    *
    >    * This code is free software; you can redistribute it and/or modify it
    > @@ -43,6 +43,7 @@
    >
    >   /**
    >    * @test
    > + * @key headful
    >    * @bug 8029253
    >    * @summary Tests asymmetric source offsets when unmanaged image is drawn to VI.
    >    *          Results of the blit to compatibleImage are used for comparison.
    >

    This change seems to get in with JDK-8221497.

    > Also, the following changeset header should work. It conforms to the guideline that the original User be preserved.
    >
    > # HG changeset patch
    > # User goetz
    > # Date 1465893899 -7200
    > #      Tue Jun 14 10:44:59 2016 +0200
    > # Node ID 980da45565c8583637a0842ba9d4eec217bdf366
    > # Parent  218c24b207fd92383fe7d880d7f7f06820cd5246
    > 8159690: [TESTBUG] Mark headful tests with @key headful.
    > Reviewed-by: simonis, alexsch
    > Contributed-by: jdowland at redhat.com

    I will fix the header before pushing this patch for Jonathan (he is not
    an author).

    Thanks,

    -Zhengyu

    >
    > Thanks,
    > Paul
    >
    > On 10/29/20, 8:06 AM, "jdk8u-dev on behalf of Zhengyu Gu" <jdk8u-dev-retn at openjdk.java.net on behalf of zgu at redhat.com> wrote:
    >
    >      Lgtm.
    >
    >      -Zhengyu
    >
    >      On 8/5/20 11:21 AM, Zhengyu Gu wrote:
    >      > Hi,
    >      >
    >      > On 7/20/20 5:35 AM, Jonathan Dowland wrote:
    >      >> Hello,
    >      >>
    >      >> Please review this backport of JDK-8159690 to 8u, for parity with
    >      >> Oracle 8u271.
    >      >>
    >      >> Bug: https://bugs.openjdk.java.net/browse/JDK-8159690
    >      >> jdk9 patch: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/980da45565c8
    >      >>
    >      >> webrev:
    >      >>      https://jmtd.net/tmp/JDK-8159690/
    >      >
    >      > Jonathan is not an author right now, so I host his webrev.
    >      > Webrev: http://cr.openjdk.java.net/~zgu/jonathan_downland/JDK-8159690/
    >      >
    >      > Thanks,
    >      >
    >      > -Zhengyu
    >      >
    >      >>
    >      >> 542/842 hunks applied directly from the original patch. 44 I resolved
    >      >> manually.  The remaining hunks are for filenames that do not exist in
    >      >> jdk8u. Those basenames are listed here:
    >      >>      https://jmtd.net/tmp/JDK-8159690/basenames_not_in_tree.txt
    >      >>
    >      >> Note that there are two different tests with the basename
    >      >> "ImageTransferTest.java", only one of which is in jdk8u.
    >      >>
    >      >>
    >      >> Thank you,
    >      >>
    >
    >




More information about the jdk8u-dev mailing list