Branch: refs/heads/main
Home: https://github.com/WebKit/WebKit
Commit: 4cfb75f6a5c326e16d2e843027dc1c125ddccc44
https://github.com/WebKit/WebKit/commit/4cfb75f6a5c326e16d2e843027dc1c125ddccc44
Author: Alex Christensen <[email protected]>
Date: 2024-11-11 (Mon, 11 Nov 2024)
Changed paths:
M Source/WebCore/platform/Site.cpp
M Source/WebCore/platform/Site.h
M Source/WebKit/UIProcess/Cocoa/WebProcessPoolCocoa.mm
M Source/WebKit/UIProcess/WebProcessPool.cpp
M Source/WebKit/UIProcess/WebProcessPool.h
M Source/WebKit/UIProcess/WebProcessProxy.cpp
Log Message:
-----------
Remove WebCore::Site default constructor
https://bugs.webkit.org/show_bug.cgi?id=282948
rdar://139666130
Reviewed by Matthew Finkel.
I added it in 286108@main to keep that relatively large change more clear that
it didn't change any behavior. This is a smaller follow-up to remove it.
Site can be constructed from a URL or a SecurityOriginData or a protocol and
domain.
Removing the empty constructor will help us keep things a little more
conceptually
clean. If there can be no site, then we want a std::optional<Site> which we
already
have in at least 1 place.
* Source/WebCore/platform/Site.cpp:
* Source/WebCore/platform/Site.h:
* Source/WebKit/UIProcess/Cocoa/WebProcessPoolCocoa.mm:
(WebKit::logProcessPoolState):
* Source/WebKit/UIProcess/WebProcessPool.cpp:
(WebKit::WebProcessPool::processForSite):
(WebKit::WebProcessPool::createWebPage):
* Source/WebKit/UIProcess/WebProcessPool.h:
* Source/WebKit/UIProcess/WebProcessProxy.cpp:
(WebKit::WebProcessProxy::didStartProvisionalLoadForMainFrame):
Canonical link: https://commits.webkit.org/286457@main
To unsubscribe from these emails, change your notification settings at
https://github.com/WebKit/WebKit/settings/notifications
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
