Details

Reviewers

tomek
bartek

Commits

rCOMM20df9c73ffe4: [web] free memory when clearing crypto store

Summary

Fixing issue described in ENG-10144#comment-16a38c06.

Tracked in ENG-10183.

We have a long-lived in-memory object here, we need to free the memory only when clearing the crypto store.

Depends on D14350

Test Plan

Logout on web app

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Feb 11 2025, 2:16 PM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptFeb 11 2025, 2:16 PM

fix typo

kamil added a child revision: D14352: [web] free memory in `createAndPersistNotificationsOutboundSession`.Feb 11 2025, 2:24 PM

Harbormaster completed remote builds in B33468: Diff 47001.Feb 11 2025, 2:35 PM

Harbormaster completed remote builds in B33469: Diff 47002.Feb 11 2025, 2:41 PM

kamil published this revision for review.Feb 11 2025, 2:44 PM

kamil edited the summary of this revision. (Show Details)

tomek accepted this revision.Feb 11 2025, 4:53 PM

This revision is now accepted and ready to land.Feb 11 2025, 4:53 PM

Closed by commit rCOMM20df9c73ffe4: [web] free memory when clearing crypto store. · Explain WhyFeb 12 2025, 4:48 PM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM20df9c73ffe4: [web] free memory when clearing crypto store.

[web] free memory when clearing crypto store
ClosedPublic
Actions

Details

Diff Detail

Event Timeline

Revision Contents
Changeset List

Diff 47073

web/shared-worker/worker/worker-crypto.js

[web] free memory when clearing crypto storeClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 47073

web/shared-worker/worker/worker-crypto.js

[web] free memory when clearing crypto store
ClosedPublic
Actions

Revision Contents
Changeset List