[webkit-changes] [WebKit/WebKit] d6da6b: testharness.js tests with complex urls do not work...

[Kimmo Kinnunen]

  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: d6da6bd550b372becfc251090ff73f3f99449d90
      https://github.com/WebKit/WebKit/commit/d6da6bd550b372becfc251090ff73f3f99449d90
  Author: Kimmo Kinnunen <kkinnunen at apple.com>
  Date:   2024-09-26 (Thu, 26 Sep 2024)

  Changed paths:
    M LayoutTests/resources/testharnessreport.js
    M Tools/WebKitTestRunner/TestInvocation.cpp

  Log Message:
  -----------
  testharness.js tests with complex urls do not work when loaded in cross-origin iframes
https://bugs.webkit.org/show_bug.cgi?id=280404
rdar://136749450

Reviewed by Charlie Wolfe and Alex Christensen.

Improve the logic to insert the runInCrossOriginFrame parameter to
the URL in case RWT sends a URL that contains search parameters or
fragments.

* LayoutTests/resources/testharnessreport.js:
* Tools/WebKitTestRunner/TestInvocation.cpp:
(WTR::addQueryParameter):
(WTR::TestInvocation::loadTestInCrossOriginIframe):
(WTR::TestInvocation::dumpWebProcessUnresponsiveness):
(WTR::TestInvocation::dumpResults):
(WTR::TestInvocation::didReceiveMessageFromInjectedBundle):
(WTR::TestInvocation::didReceiveSynchronousMessageFromInjectedBundle):
(WTR::TestInvocation::runUISideScript):
(WTR::TestInvocation::waitToDumpWatchdogTimerFired):

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



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