Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 392e336708452baf15ee7827968d2860a43391a5
      
https://github.com/WebKit/WebKit/commit/392e336708452baf15ee7827968d2860a43391a5
  Author: Alex Christensen <achristen...@apple.com>
  Date:   2025-04-01 (Tue, 01 Apr 2025)

  Changed paths:
    M Source/WebKit/WebProcess/WebPage/WebFrame.cpp
    M Source/WebKit/WebProcess/WebPage/WebFrame.h
    M Source/WebKit/WebProcess/WebPage/WebPage.cpp
    M Tools/TestWebKitAPI/Tests/WebKitCocoa/SiteIsolation.mm

  Log Message:
  -----------
  Omit CertificateInfo from FrameInfoData unless requested
https://bugs.webkit.org/show_bug.cgi?id=290877
rdar://148331294

Reviewed by Per Arne Vollan.

In 290521@main I added a CertificateInfo to FrameInfoData, but that was a
performance regression.  It turns out CertificateInfo for some sites are
expensive to serialize and deserialize, and FrameInfoData is sent in many
IPC messages during loading and otherwise.  This omits the CertificateInfo
unless WKWebView._frames has requested it, which should resolve the perf bug.

* Source/WebKit/WebProcess/WebPage/WebFrame.cpp:
(WebKit::WebFrame::info const):
* Source/WebKit/WebProcess/WebPage/WebFrame.h:
* Source/WebKit/WebProcess/WebPage/WebPage.cpp:
(WebKit::WebPage::getFrameInfo):
* Tools/TestWebKitAPI/Tests/WebKitCocoa/SiteIsolation.mm:
(TestWebKitAPI::TEST(SiteIsolation, FrameServerTrust)):

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



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

Reply via email to