Bug 213881 – [GTK][WPE] Enable storageAccess tests

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Carlos Garcia Campos]

They are currently skipped by the main TestExpectations file, but we should run them now that we support ITP.

The current status is:

68 tests ran as expected, 13 didn't:

Regressions: Unexpected text-only failures (13)
  http/tests/storageAccess/deny-due-to-no-interaction-under-general-third-party-cookie-blocking-ephemeral.html [ Failure ]
  http/tests/storageAccess/deny-storage-access-under-opener-ephemeral.html [ Failure ]
  http/tests/storageAccess/deny-storage-access-under-opener-if-auto-dismiss-ephemeral.html [ Failure ]
  http/tests/storageAccess/deny-with-prompt-does-not-preserve-gesture-database.html [ Failure ]
  http/tests/storageAccess/deny-with-prompt-does-not-preserve-gesture.html [ Failure ]
  http/tests/storageAccess/grant-storage-access-under-opener-at-popup-user-gesture-ephemeral.html [ Failure ]
  http/tests/storageAccess/has-storage-access-under-general-third-party-cookie-blocking-with-cookie-ephemeral.html [ Failure ]
  http/tests/storageAccess/request-and-grant-access-then-detach-should-not-have-access-database.html [ Failure ]
  http/tests/storageAccess/request-and-grant-access-then-detach-should-not-have-access.html [ Failure ]
  http/tests/storageAccess/request-and-grant-access-then-navigate-cross-site-should-not-have-access-database.html [ Failure ]
  http/tests/storageAccess/request-and-grant-access-then-navigate-cross-site-should-not-have-access.html [ Failure ]
  http/tests/storageAccess/request-and-grant-access-then-navigate-same-site-should-have-access-database.html [ Failure ]
  http/tests/storageAccess/request-and-grant-access-then-navigate-same-site-should-have-access.html [ Failure ]

Some of those failures are also skipped in mac-wk2, but we need to investigate the others.

[Carlos Garcia Campos]

Most of the *-ephemeral.html tests that were failing are passing now after r263895.

http/tests/storageAccess/deny-with-prompt-does-not-preserve-gesture.html
http/tests/storageAccess/deny-with-prompt-does-not-preserve-gesture-database.html

These are skipped in mac, but without a bug number, so I don't know why.

http/tests/storageAccess/has-storage-access-true-if-third-party-has-cookies-ephemeral-database.html
http/tests/storageAccess/has-storage-access-true-if-third-party-has-cookies-ephemeral.html

These are bug #208400.

With those skipped and patch from bug #213502 applied the situation is:

252 tests ran as expected, 4 didn't:

Regressions: Unexpected text-only failures (4)
  http/tests/storageAccess/request-and-grant-access-then-navigate-cross-site-should-not-have-access-database.html [ Failure ]
  http/tests/storageAccess/request-and-grant-access-then-navigate-cross-site-should-not-have-access.html [ Failure ]
  http/tests/storageAccess/request-and-grant-access-then-navigate-same-site-should-have-access-database.html [ Failure ]
  http/tests/storageAccess/request-and-grant-access-then-navigate-same-site-should-have-access.html [ Failure ]

These are failing because tests are using OnlyAccordingToPerDomainPolicy as ThirdPartyCookieBlockingMode, so the cookie accept policy is kept as no-third-party. Cookies granted by storage access are still rejected by the cookie accept policy. This should only happen in tests, because we always use All as ThirdPartyCookieBlockingMode.

[Carlos Garcia Campos]

Created attachment 403582 [details]
Patch

[Carlos Garcia Campos]

Committed r263965: <https://trac.webkit.org/changeset/263965>