Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 5e66420b6d442beb27ff645a7520f40dff0b52c4
      
https://github.com/WebKit/WebKit/commit/5e66420b6d442beb27ff645a7520f40dff0b52c4
  Author: Wenson Hsieh <[email protected]>
  Date:   2024-10-28 (Mon, 28 Oct 2024)

  Changed paths:
    M Tools/TestWebKitAPI/Tests/WebKitCocoa/ElementTargetingTests.mm

  Log Message:
  -----------
  REGRESSION (285706@main): 
ElementTargeting.CountVisibilityAdjustmentsAfterNavigatingBack hits a flaky 
debug assertion
https://bugs.webkit.org/show_bug.cgi?id=282194

Reviewed by Abrar Rahman Protyasha.

The changes to `Tools/TestWebKitAPI/Tests/WebKitCocoa/nested-frames.html` in 
285706@main caused this
API test to sometimes hit an assert in debug builds. This exposed a latent 
issue with this test,
which attempts to load two test pages (`element-targeting-1.html` and 
`element-targeting-2.html`)
back-to-back and then navigate back once, but doesn't attempt to wait for the 
first page (and
importantly, all of its subframes) to finish loading before moving on to the 
next page.

Fix this by waiting for `subframeLoaded` to become true.

* Tools/TestWebKitAPI/Tests/WebKitCocoa/ElementTargetingTests.mm:
(TestWebKitAPI::TEST(ElementTargeting, 
CountVisibilityAdjustmentsAfterNavigatingBack)):

Canonical link: https://commits.webkit.org/285791@main



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to