On Thu, 14 Mar 2024 21:27:47 GMT, Phil Race <p...@openjdk.org> wrote:
> After some experimentation/thought I think it best to delete this applet test. > > It is supposed to be a manual test but is missing the manual keyword so > as far as I can tell it has never been usefully run. Except perhaps a few > times > when first being verified. > When I run it in jtreg a window popups up, disappears and the test is passed > Contradicting instructions in the test that tell you it has to be manual > because > you need to run it *in the browser* not jtreg. > > Well, we don't support browser applets any more. > And the scenario it was testing was explicitly a case that happened with the > browser. > > And to cap it all, I see no way for you to tell jtreg that it passed or > failed - because > you run it in the browser. > > So it is manual in the sense that it is NOT a jtreg test and so it should not > even have > an @test tag. > > These are more than enough reasons to just delete it. This pull request has now been integrated. Changeset: 09439374 Author: Phil Race <p...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/09439374edec8178fc2a0494bb96a8a24384c2a3 Stats: 154 lines in 2 files changed: 0 ins; 154 del; 0 mod 8328218: Delete test java/awt/Window/FindOwner/FindOwner.html Reviewed-by: psadhukhan ------------- PR: https://git.openjdk.org/jdk/pull/18314