Do not try to swap navigation as we're not using PSON

magomez · pgorszkowski-igalia · commit ee25625bc407 · 2024-05-02T08:14:00.000+02:00
diff --git a/Source/WebKit/NetworkProcess/NetworkResourceLoader.cpp b/Source/WebKit/NetworkProcess/NetworkResourceLoader.cpp
@@ -826,6 +826,9 @@ void NetworkResourceLoader::processClearSiteDataHeader(const WebCore::ResourceRe
 
 static BrowsingContextGroupSwitchDecision toBrowsingContextGroupSwitchDecision(const std::optional<CrossOriginOpenerPolicyEnforcementResult>& currentCoopEnforcementResult)
 {
+    // PSON is disabled, so return a value that doesn't cause the navigation to be swapped to a new WebProcess.
+    return BrowsingContextGroupSwitchDecision::StayInGroup;
+
     if (!currentCoopEnforcementResult || !currentCoopEnforcementResult->needsBrowsingContextGroupSwitch)
         return BrowsingContextGroupSwitchDecision::StayInGroup;
     if (currentCoopEnforcementResult->crossOriginOpenerPolicy.value == CrossOriginOpenerPolicyValue::SameOriginPlusCOEP)

Original file line number	Diff line number	Diff line change
`@@ -826,6 +826,9 @@ void NetworkResourceLoader::processClearSiteDataHeader(const WebCore::ResourceRe`
`826`	`826`
`827`	`827`	`static BrowsingContextGroupSwitchDecision toBrowsingContextGroupSwitchDecision(const std::optional<CrossOriginOpenerPolicyEnforcementResult>& currentCoopEnforcementResult)`
`828`	`828`	`{`
	`829`	`+ // PSON is disabled, so return a value that doesn't cause the navigation to be swapped to a new WebProcess.`
	`830`	`+ return BrowsingContextGroupSwitchDecision::StayInGroup;`
	`831`	`+`
`829`	`832`	`if (!currentCoopEnforcementResult \|\| !currentCoopEnforcementResult->needsBrowsingContextGroupSwitch)`
`830`	`833`	`return BrowsingContextGroupSwitchDecision::StayInGroup;`
`831`	`834`	`if (currentCoopEnforcementResult->crossOriginOpenerPolicy.value == CrossOriginOpenerPolicyValue::SameOriginPlusCOEP)`