Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: b41236839521e892d553b7dcc6795ac7d61b3c74
      
https://github.com/WebKit/WebKit/commit/b41236839521e892d553b7dcc6795ac7d61b3c74
  Author: Roberto Rodriguez <[email protected]>
  Date:   2026-05-04 (Mon, 04 May 2026)

  Changed paths:
    M LayoutTests/platform/ios-site-isolation/TestExpectations
    M LayoutTests/platform/mac-site-isolation/TestExpectations
    M 
LayoutTests/security/contentSecurityPolicy/plugins-types-allows-youtube-plugin-replacement-expected.txt
    M 
LayoutTests/security/contentSecurityPolicy/plugins-types-allows-youtube-plugin-replacement.html
    M Tools/WebKitTestRunner/TestController.cpp

  Log Message:
  -----------
  [Site Isolation] Update expected output for 
plugins-types-allows-youtube-plugin-replacement.html
https://bugs.webkit.org/show_bug.cgi?id=313851
rdar://176052705

Reviewed by Megan Gardner and Ryosuke Niwa.

The number of deprecation console messages differ between non-SI and SI 
configurations because the
youtube.com iframe navigation is canceled at different loading stages, 
producing different numbers of
about:blank documents that inherit and re-parse the parent CSP where each 
re-parse fires the deprecation
warning again. The current expected output only matches the non-SI case.

Since the test only needs to verify the plugin replacement iframe is created, 
suppress the console
messages from the output so both configurations pass.

Also update TestController.cpp's SI external URL blocking path to check 
dumpJSConsoleLogInStdErr. Without it
the "Blocked access" message leaks into test output under SI even when console 
messages are suppressed,
because this path writes directly to the output rather than going through the 
console pipeline.

* LayoutTests/platform/ios-site-isolation/TestExpectations:
* LayoutTests/platform/mac-site-isolation/TestExpectations:
* 
LayoutTests/security/contentSecurityPolicy/plugins-types-allows-youtube-plugin-replacement-expected.txt:
* 
LayoutTests/security/contentSecurityPolicy/plugins-types-allows-youtube-plugin-replacement.html:
* Tools/WebKitTestRunner/TestController.cpp:

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



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

Reply via email to