[webkit-changes] [WebKit/WebKit] c741b1: REGRESSION(266644 at main):[ macOS WK1 ] report-origi...

Brent Fulgham noreply at github.com
Wed May 8 18:38:20 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: c741b1a50100c46daa4c10c0c06ed3bb1a65c35d
      https://github.com/WebKit/WebKit/commit/c741b1a50100c46daa4c10c0c06ed3bb1a65c35d
  Author: Brent Fulgham <bfulgham at apple.com>
  Date:   2024-05-08 (Wed, 08 May 2024)

  Changed paths:
    M LayoutTests/platform/mac-wk1/TestExpectations
    A LayoutTests/platform/mac-wk1/imported/w3c/web-platform-tests/content-security-policy/reporting/report-original-url-on-mixed-content-frame.https.sub-expected.txt

  Log Message:
  -----------
  REGRESSION(266644 at main):[ macOS WK1 ] report-original-url-on-mixed-content-frame.https.sub.html is a constant text failure
https://bugs.webkit.org/show_bug.cgi?id=266047
<rdar://problem/119349300>

Reviewed by Tim Nguyen.

The original result for  LayoutTests/imported/w3c/web-platform-tests/content-security-policy/reporting/report-original-url-on-mixed-content-frame.https.sub-expected.txt
was failure on all platforms.

Ryan's work in 266644 at main progressed this test case in modern WebKit, and he checked that expectation in.

Unfortunately, WebKitLegacy relies on a PingHandle implementation that cannot authenticate against https
(which is a missing feature in the entire deprecated WebKitLegacy platform for the old PingHandle implementation).
We do to intend to invest time working on implementing such a feature in WebKitLegacy, so we expect this
test to fail.

This patch lands a new WebKitLegacy test expectation to document the expected behavior. Note that the console
output on WebKitLegacy shows that the load was blocked properly in WebKitLegacy. It is only the Reporting API
message that fails in this test, simply because WebKitLegacy cannot handle the mixture of an insecure main
page sending reports to an secure reporting target.

* LayoutTests/platform/mac-wk1/TestExpectations:
* LayoutTests/platform/mac-wk1/imported/w3c/web-platform-tests/content-security-policy/reporting/report-original-url-on-mixed-content-frame.https.sub-expected.txt: Added.

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



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


More information about the webkit-changes mailing list