1574690 - Adjust wpt metadata for COOP and COEP tests

Status: RESOLVED FIXED

(Core :: DOM: Networking, task, P2)

Description

[Junior [inactive]]

Please see bug 1564430 comment 3, we need to turn on prefs.
Also some old metadata should be removed.

Comment 1

[Junior [inactive]]

Local experiments show we might fails the following tests with the current timeout in tests:

Similar to Bug 1576153

/html/cross-origin-embedder-policy/require-corp.https.html                                                                                                         
  FAIL "require-corp" top-level: navigating a frame from "require-corp" to "none" should fail - assert_equals: expected null but got Document node with 2 children 
bc.onmessage</<@https://web-platform.test:8443/html/cross-origin-embedder-policy/require-corp.https.html:34:20                                                     

Bug 1574612

/html/cross-origin-embedder-policy/none.https.html
  FAIL "require-corp" top-level noopener popup: navigating to "none" should succeed - assert_true: expected true got false
@https://web-platform.test:8443/html/cross-origin-embedder-policy/none.https.html:75:16

I didn't address Bug 1575993, not sure if it's fixed or not.

Comment 2

[Junior [inactive]]

Attachment: Bug 1574690 - P1 Adjust wpt metadata for COOP and COEP tests

Comment 3

[Junior [inactive]]

one timeout intermittent
https://treeherder.mozilla.org/#/jobs?repo=try&revision=bdb9bd198fe7655ee63a6236e30e4c734b06ae61

Comment 4

[Junior [inactive]]

IMO they simply don't have enough time to finish 18 popups. Let's see if treeherder is happier
https://treeherder.mozilla.org/#/jobs?repo=try&revision=8afb975d6c206c6a7764609f648d7391d1995ebf

Comment 5

[Junior [inactive]]

Attachment: Bug 1574690 - require longer timeout for coop test which opens 18 popups

Comment 6

[Junior [inactive]]

Attachment: Bug 1574690 - P2 COOP intermittent timeout adjustment

We fail to release component manager in content process, which should has nothing to do with COOP implemetation

Comment 7

[Junior [inactive]]

Attachment: Bug 1574690 - P3 longer timeout to ease intermittent COEP test

This might be necessary since we fail to load the second blank page in 500 ms all the times

Comment 8

[Junior [inactive]]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=926867e06480cab64b16a4a2376a367b59f3482b

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a6a1449c2a20
P1 Adjust wpt metadata for COOP and COEP tests r=annevk
https://hg.mozilla.org/integration/autoland/rev/574669f5c521
P2 COOP intermittent timeout adjustment r=valentin
https://hg.mozilla.org/integration/autoland/rev/a424a0179974
P3 longer timeout to ease intermittent COEP test r=annevk

Comment 10

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/19029 for changes under testing/web-platform/tests

Comment 11

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Can't merge web-platform-tests PR due to failing upstream checks:
Github PR https://github.com/web-platform-tests/wpt/pull/19029
* Taskcluster (pull_request) (https://tools.taskcluster.net/task-group-inspector/#/Ng60AmdNQmmgeRMw95eRCw)

Comment 12

[Junior [inactive]]

I'm not sure what version of firefox wpt-firefox-nightly-stability ran with.
Looks like it didn't toggle the pref which comment 2 did, but it relied on the new .ini which EXPECT: FAIL is removed.

Maybe sync to again again after next nightly?

Comment 13

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/a6a1449c2a20
https://hg.mozilla.org/mozilla-central/rev/574669f5c521
https://hg.mozilla.org/mozilla-central/rev/a424a0179974

Comment 14

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Upstream PR merged by jgraham

Comment 15

[Junior [inactive]]

Thanks jgraham!