Remove OriginStorageManager if it's not in use

• Component: New Bugs
• Status: RESOLVED FIXED

[Sihui Liu]

...

[Sihui Liu]

Created attachment 450674 [details]
Patch

[Sihui Liu]

Created attachment 450679 [details]
Patch

[Darin Adler]

Comment on attachment 450679 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=450679&action=review

> Source/WebKit/NetworkProcess/storage/NetworkStorageManager.cpp:181
> +        HashSet<WebCore::ClientOrigin> originsToRemove;
> +        for (auto& [origin, manager] : m_localOriginStorageManagers) {
> +            manager->connectionClosed(connection);
> +            if (!manager->isActive())
> +                originsToRemove.add(origin);
> +        }
> +
> +        for (auto origin : originsToRemove)
> +            m_localOriginStorageManagers.remove(origin);

HashSet has a removeIf function that can do this sort of thing much more efficiently. We should use it here.

[Sihui Liu]

Created attachment 450705 [details]
Patch for landing

[EWS]

Committed r289012 (246721@main): <https://commits.webkit.org/246721@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 450705 [details].

[Radar WebKit Bug Importer]

<rdar://problem/88406602>