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

Hohensee, Paul hohensee at amazon.com
Thu Oct 29 15:37:29 UTC 2020


Hi, Jonathan, just a couple of comments.

Looks like you're missing a file in your patch, namely

test/java/awt/image/DrawImage/IncorrectUnmanagedImageSourceOffset.java

For that file, the original changeset has:

diff --git a/test/java/awt/image/DrawImage/IncorrectUnmanagedImageSourceOffset.java b/test/java/awt/image/DrawImage/IncorrectUnmanagedImageSourceOffset.java
--- a/test/java/awt/image/DrawImage/IncorrectUnmanagedImageSourceOffset.java
+++ b/test/java/awt/image/DrawImage/IncorrectUnmanagedImageSourceOffset.java
@@ -1,5 +1,5 @@
 /*
- * 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.

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

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