Issue 1509740 in chromium: [WPT] New failures introduced in external/wpt/content-security-policy by import https://crrev.com/c/5102648

[wpt-autoroller via monorail]

Status: Untriaged
Owner: ----
CC: securi...@chromium.org, arthurs...@chromium.org, antonio...@chromium.org
Labels: Test-WebTest
Components: Blink>SecurityFeature>ContentSecurityPolicy
Pri: 3
Type: Bug

New issue 1509740 by wpt-aut...@chops-service-accounts.iam.gserviceaccount.com: [WPT] New failures introduced in external/wpt/content-security-policy by import https://crrev.com/c/5102648
https://bugs.chromium.org/p/chromium/issues/detail?id=1509740

WPT import https://crrev.com/c/5102648 introduced new failures in external/wpt/content-security-policy:

List of new failures:
[ Mac-mac12 ] external/wpt/content-security-policy/reporting/report-same-origin-with-cookies.html new failing tests: https://crrev.com/c/5102648/4/third_party/blink/web_tests/platform/mac-mac12/external/wpt/content-security-policy/reporting/report-same-origin-with-cookies-expected.txt

Expectations or baseline files [0] have been automatically added for the failing results to keep the bots green. Please investigate the new failures and triage as appropriate.

Upstream changes imported:
https://github.com/web-platform-tests/wpt/compare/b9ac113d1b07d279208c1c6150e902181967d021...d51b3698150ed6883df3f7ee2f11bbca4567e40e
Grid items with align/justify-self should participate in baseline alginment: https://github.com/web-platform-tests/wpt/commit/d51b3698150ed6883df3f7ee2f11bbca4567e40e
Revert "Fix NOTREACHED in layoutTheme::DefaultSystemColor": https://github.com/web-platform-tests/wpt/commit/e883ed3101588a604d6d77b9bbaa44d748692b23
[FedCM] Make accountHint required: https://github.com/web-platform-tests/wpt/commit/c41d8c520683379b1fbf99cbce610fe80f63988e
[A11y] Simplification -- don't create tree holes for layout updates: https://github.com/web-platform-tests/wpt/commit/41414fff6d5b33b7b54ca248c8206bef75566fa2
Add check for invalid origins - requestStorageAccessFor: https://github.com/web-platform-tests/wpt/commit/398e3d8f9967585cf73c6d479003cebf152d152b
IndexedDB: run just a small subset of nested-cloning tests on MSAN.: https://github.com/web-platform-tests/wpt/commit/24b2021980addfbd3ab52a10a7df22493deda61e
[FedCM] Do not crash for pending disconnect/getUserInfo: https://github.com/web-platform-tests/wpt/commit/29cdefab41a941e0fce230c5475ee1459a390a02
[FLEDGE] Calculate trusted_bidding_signals_slot_size_param: https://github.com/web-platform-tests/wpt/commit/75dd6dec07645f9848668711ae335bf447f0b7e1
Make transform builder use resolvable ComputeNumber for scale(): https://github.com/web-platform-tests/wpt/commit/e4e8e8c26d7afd7a221c5db0fa44ed65cff23e06
Revert "[keepalive-migration] Add fieldtrial testing config": https://github.com/web-platform-tests/wpt/commit/7b0ac953e8fcaccbf5e71ff046e68bc7585a8a40
[css-variables] Count “--foo: initial” as a style declaration.: https://github.com/web-platform-tests/wpt/commit/ec42cbde465a2cae1c641c464bf52dcf6544f805
Missing ::first-letter style update for size queries: https://github.com/web-platform-tests/wpt/commit/94b37f7624fb523c15cdd581755a2556ddd2b65b
Add noto fonts to the Docker image: https://github.com/web-platform-tests/wpt/commit/00dee68a67459b04b971f928c7360cb1be036613
[keepalive-migration] Add fieldtrial testing config: https://github.com/web-platform-tests/wpt/commit/e98188eff115b25d1b1f6bc4122bcd338dc65326
Move popover=hint related code out of non-tentative test: https://github.com/web-platform-tests/wpt/commit/fcd111f4f95c9dae1d0dd7f938fe427699534bc0
Avoid bad relayout roots inside pagination.: https://github.com/web-platform-tests/wpt/commit/d28d3aae7f6bcf91a8c3f1db12852cb17511d2f3
Split out anchor-attribute related stuff from popover test: https://github.com/web-platform-tests/wpt/commit/e905d6af9e32b9cfa514f8950c6ec82e3d59e4ad
Make ic-width consistent regardless of text orientation.: https://github.com/web-platform-tests/wpt/commit/d1bb35dae8f4a383b45e17558f4631a322cc8a54
Add waitForCompositorReady to scroll-behavior-* tests.: https://github.com/web-platform-tests/wpt/commit/ae1e786b4a77e4256a6873b1fbeedf48c4437831
LoAF: for no-cors scripts, hide executionStart: https://github.com/web-platform-tests/wpt/commit/b675788b7ffa130c95611124da4b4d964e46b23a
[css-scroll-snap-2] Implement snapchanging for user scrolls: https://github.com/web-platform-tests/wpt/commit/63bf12229d8b71f6f4abff0fd96b8418410797a7
Allow cross-origin subframes to send automatic beacons.: https://github.com/web-platform-tests/wpt/commit/458d51295af838f04608bda0dc912aaf88bc1de3
wpt: Update some security-features scripts to run under Python 3.: https://github.com/web-platform-tests/wpt/commit/8b2ff8f91555ee3cda84ad9bca665d960c4769e7
Fix memory leak in mousemove_prevent_default_action.: https://github.com/web-platform-tests/wpt/commit/730060196d22566dd233cdbfda9c0c807564ff16

[0]: https://chromium.googlesource.com/chromium/src/+/HEAD/docs/testing/web_test_expectations.md

This bug was filed automatically due to a new WPT test failure for which you are marked an OWNER. If you do not want to receive these reports, please add "wpt { notify: NO }" to the relevant DIR_METADATA file.

--
You received this message because:
1. You were specifically CC'd on the issue

You may adjust your notification preferences at:
https://bugs.chromium.org/hosting/settings

Reply to this email to add a comment or make updates.

[vogelheim via monorail]

Comment #1 on issue 1509740 by voge...@google.com: [WPT] New failures introduced in external/wpt/content-security-policy by import https://crrev.com/c/5102648
https://bugs.chromium.org/p/chromium/issues/detail?id=1509740#c1

[OWP sec triage] The test complains about a cookie not being reported. I can't see any obvious change in the WPT tests, so I'm guessing this is a behaviour change in Chrome.

WPT suggests that FF doesn't do this either, but Chrome and Safari do:
https://wpt.fyi/results/content-security-policy/reporting/report-same-origin-with-cookies.html?label=master&label=experimental&aligned&q=content-security-policy%2Freporting%2Freport-same-origin-with-cookies

It's weird that this expectation change would occur with exporting WPT tests, and without any related WPT test changing. I wonder whether this is some flag about cookie handling being flipped.

[antoniosartori via monorail]

Updates:
Owner: antonio...@chromium.org
Status: Assigned

Comment #2 on issue 1509740 by antonio...@chromium.org: [WPT] New failures introduced in external/wpt/content-security-policy by import https://crrev.com/c/5102648
https://bugs.chromium.org/p/chromium/issues/detail?id=1509740#c2

The test is flaky, not failing. Flakiness started yesterday, so this is something we should investigate. I'll have a look.

[antoniosartori via monorail]

Comment #3 on issue 1509740 by antonio...@chromium.org: [WPT] New failures introduced in external/wpt/content-security-policy by import https://crrev.com/c/5102648
https://bugs.chromium.org/p/chromium/issues/detail?id=1509740#c3

It's not clear to me why, but a bisection suggests that this is related to https://chromium-review.googlesource.com/c/chromium/src/+/5101871 (which is a reland of a revert).

[antoniosartori via monorail]

Comment #4 on issue 1509740 by antonio...@chromium.org: [WPT] New failures introduced in external/wpt/content-security-policy by import https://crrev.com/c/5102648
https://bugs.chromium.org/p/chromium/issues/detail?id=1509740#c4

Flakiness tracked in crbug.com/1509997

[antoniosartori via monorail]

Updates:
Status: Fixed

Comment #5 on issue 1509740 by antonio...@chromium.org: [WPT] New failures introduced in external/wpt/content-security-policy by import https://crrev.com/c/5102648
https://bugs.chromium.org/p/chromium/issues/detail?id=1509740#c5

(No comment was entered for this change.)