We still use `topLevelPrincipal` from LoadInfo to get `partitionPrincipal` at [here](https://searchfox.org/mozilla-central/rev/35b97af64a55d1d30caa4d6e9fabc1a7fbabc509/toolkit/components/antitracking/StoragePrincipalHelper.cpp#61). But it is only used when we can't partitionKey from CookieJarSetting in some special case, so probably in most of the cases, the value of `topLevelPrincipal` doesn't matter. I think the right solution now is to find the cases we don't set partitionKey, and then remove `TopLevelPrincipal` from `LoadInfo` entirely to prevent using it in the future. Because this value doesn't work in Fission anyway.
Bug 1618557 Comment 7 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
We still use `topLevelPrincipal` from LoadInfo to get `partitionPrincipal` at [here](https://searchfox.org/mozilla-central/rev/35b97af64a55d1d30caa4d6e9fabc1a7fbabc509/toolkit/components/antitracking/StoragePrincipalHelper.cpp#61). But it is only used when we can't partitionKey from CookieJarSetting in some special cases, so in most of the cases, the value of `topLevelPrincipal` doesn't matter. I think the right solution now is to find the cases we don't set partitionKey, and then remove `TopLevelPrincipal` from `LoadInfo` entirely to prevent using it in the future. Because this value doesn't work in Fission anyway.