RFR: 8335548: testCookieEnabled fails with WebKit 619.1

Fri Jul 19 18:28:39 UTC 2024

On Fri, 19 Jul 2024 01:11:24 GMT, Jay Bhaskar <jbhaskar at openjdk.org> wrote:

> Issue : CookieEnable method returns false when WebView url is empty
> Solution: Use a dummy file URI in WebView before testing cookie.

I confirm that the test now passes. I left one comment on the html file you added and will reapprove if you change it.

modules/javafx.web/src/test/resources/test/html/cookie.html line 4:

> 2: <html>
> 3: <head>
> 4:     <meta http-equiv="Refresh" content="100; URL=ipsum.html" />

The redirect to another html file seems unnecessary. Is there a reason you added it? If I locally remove the `<head>...</head>` block the still passes.

-------------

Marked as reviewed by kcr (Lead).

PR Review: https://git.openjdk.org/jfx/pull/1513#pullrequestreview-2188920799
PR Review Comment: https://git.openjdk.org/jfx/pull/1513#discussion_r1684800629