Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 0351e7b861f3174aafbb4da98d4cd46012163e39
      
https://github.com/WebKit/WebKit/commit/0351e7b861f3174aafbb4da98d4cd46012163e39
  Author: Roberto Rodriguez <[email protected]>
  Date:   2026-02-05 (Thu, 05 Feb 2026)

  Changed paths:
    M LayoutTests/fast/dom/DOMURL/url-very-large-password-no-crash.html
    M LayoutTests/fast/dom/DOMURL/url-very-large-username-no-crash.html
    M LayoutTests/platform/ios-site-isolation/TestExpectations
    M LayoutTests/platform/mac-site-isolation/TestExpectations

  Log Message:
  -----------
  [Site Isolation] Fix fast/dom/DOMURL/url-very-large-password-no-crash.html 
test failure
https://bugs.webkit.org/show_bug.cgi?id=307010
rdar://169663801

Reviewed by Sihui Liu.

Tests url-very-large-password-no-crash.html and 
url-very-large-username-no-crash.html
called testRunner.notifyDone() before js-test-post.js loaded. This caused the 
test harness
to capture output immediately, missing the final test assertions that 
js-test-post.js writes
("PASS successfullyParsed is true" and "TEST COMPLETE"). The race condition was 
more frequent
in Site Isolation due to cross-process timing delays.

Remove the unnecessary waitUntilDone/notifyDone calls. The tests now run 
synchronously and
let js-test-post.js handle completion automatically.

* LayoutTests/fast/dom/DOMURL/url-very-large-password-no-crash.html:
* LayoutTests/fast/dom/DOMURL/url-very-large-username-no-crash.html:
* LayoutTests/platform/ios-site-isolation/TestExpectations:
* LayoutTests/platform/mac-site-isolation/TestExpectations:

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



To unsubscribe from these emails, change your notification settings at 
https://github.com/WebKit/WebKit/settings/notifications

Reply via email to