Closed Bug 1770875 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 34179 - Revert "Reland "Anonymous iframe: regression test with COI.""

Categories

(Testing :: web-platform-tests, task, P4)

task

Tracking

(firefox102 fixed)

RESOLVED FIXED
102 Branch
Tracking Status
firefox102 --- fixed

People

(Reporter: mozilla.org, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 34179 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/34179
Details from upstream follow.

Alan Cutter <alancutter@chromium.org> wrote:

Revert "Reland "Anonymous iframe: regression test with COI.""

This reverts commit 5eac39ac7d44f2e8f752729dc19484ad9fabe6b7.

Reason for revert:
New test is failing on all platforms except ASAN.
https://test-results.appspot.com/dashboards/flakiness_dashboard.html#testType=blink_web_tests&tests=embedding.tentative.https.window.html
Variants:
external/wpt/html/anonymous-iframe/embedding.tentative.https.window.html?7-8
external/wpt/html/anonymous-iframe/embedding.tentative.https.window.html?9-10

Original change's description:

Reland "Anonymous iframe: regression test with COI."

This is a reland of commit ab76764559964056dd98d168273e1ec7dfd938f3
https://chromium-review.googlesource.com/c/chromium/src/+/3653848

It was reverted in:
https://chromium-review.googlesource.com/c/chromium/src/+/3660186

This introduce a regression test. The test "Crash" in debug mode
and "Failure" in release mode. I forgot to add "Failure" too in
the TestExpectation file.

Original change's description:

Anonymous iframe: regression test with COI.

When embedding inside a cross-origin-isolated page, the
NavigationRequest::CoopCoepSanityCheck() throw an error in debug mode,
and causes the navigation to never complete in release mode.

This patch adds regressions tests.

Bug: 1314369
Change-Id: Ia2273c58fb034f21152d71e211e311f71e25011d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3653848
Reviewed-by: Arthur Hemery \<ahemery@chromium.org>
Commit-Queue: Arthur Sonzogni \<arthursonzogni@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1006363}

Bug: 1314369
Change-Id: I06f5d1068691be17994fd4c33aa688f4f1365e13
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3660170
Auto-Submit: Arthur Sonzogni \<arthursonzogni@chromium.org>
Reviewed-by: Maksim Moskvitin \<mmoskvitin@google.com>
Commit-Queue: Maksim Moskvitin \<mmoskvitin@google.com>
Commit-Queue: Arthur Sonzogni \<arthursonzogni@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1006417}

Bug: 1314369
Change-Id: I373202b079cbe193ade752189abea2e93fe64682
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3661882
Bot-Commit: Rubber Stamper \<rubber-stamper@appspot.gserviceaccount.com>
Auto-Submit: Alan Cutter \<alancutter@chromium.org>
Commit-Queue: Rubber Stamper \<rubber-stamper@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/heads/main@{#1006728}

PR 34179 applied with additional changes from upstream: 4ee5b8ea5f6af11164acc76ebb83ef4a583a2827, 13a09a654ef631054aeb332d91e64d090f349ef7, b091a59bf1d2de4de0239f6bf677ec490a97e11c
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/42c49a5d3e38
[wpt PR 34179] - Revert "Reland "Anonymous iframe: regression test with COI."", a=testonly
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 102 Branch
You need to log in before you can comment on or make changes to this bug.