Closed Bug 1824749 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 39214 - Revert "Deflake permission observation WPT by ignoring the event target"

Categories

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

task

Tracking

(firefox114 fixed)

RESOLVED FIXED
114 Branch
Tracking Status
firefox114 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Chris Fredrickson <cfredric@chromium.org> wrote:

Revert "Deflake permission observation WPT by ignoring the event target"

This reverts commit 90d9a1b12b4fafec7c3c843b231fd4ed24afe88a.

Reason for revert: flakiness dashboard indicates this test is still flaky even with this change, so this CL was not successful.

Original change's description:

Deflake permission observation WPT by ignoring the event target

This just hides the underlying problem, that the event is being fired
at the wrong target sometimes (if my hypothesis is correct, anyway).
But it will fix the flake, so we don't have to disable this test or
ignore its status.

Bug: 1427214
Change-Id: Ic97acb53d95077264e34253adc25a125b3ce1685
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4368364
Commit-Queue: Chris Fredrickson \<cfredric@chromium.org>
Auto-Submit: Chris Fredrickson \<cfredric@chromium.org>
Reviewed-by: Johann Hofmann \<johannhof@chromium.org>
Commit-Queue: Johann Hofmann \<johannhof@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1121755}

Bug: 1427214
Change-Id: Ia45f4ebf5c12c539303f816004f9a265ecb0b4d2

Reviewed-on: https://chromium-review.googlesource.com/4374078
WPT-Export-Revision: e76b9bbcf7c51a859b1c835f57a5f2c948c92ee7

Test result changes from PR not available.
Test result changes from PR not available.
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/753500216fa8 [wpt PR 39214] - Revert "Deflake permission observation WPT by ignoring the event target", a=testonly
Test result changes from PR not available.
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 114 Branch
You need to log in before you can comment on or make changes to this bug.