[web] clear old `cryptoStore` before creating new one
ClosedPublic
Actions

Authored by kamil on Mar 20 2025, 4:59 AM.

Details

Reviewers

bartek
tomek
ashoat

Commits

rCOMM941b4e92c37c: [web] clear old `cryptoStore` before creating new one

Summary

Some context in ENG-10405.

I found this issue using this PR.

Depends on D14464

Test Plan

Used memory is the same after INITIALIZE_CRYPTO_ACCOUNT request

Diff Detail

Repository

rCOMM Comm

Branch

leaks-2

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

kamil created this revision.Mar 20 2025, 4:59 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptMar 20 2025, 4:59 AM

kamil added a child revision: D14483: [web] clear old Olm session before creating new one.Mar 20 2025, 5:02 AM

Harbormaster completed remote builds in B33790: Diff 47494.Mar 20 2025, 5:31 AM

kamil published this revision for review.Mar 20 2025, 5:44 AM

ashoat accepted this revision.Mar 20 2025, 5:50 AM

This revision is now accepted and ready to land.Mar 20 2025, 5:50 AM

Closed by commit rCOMM941b4e92c37c: [web] clear old `cryptoStore` before creating new one (authored by kamil). · Explain WhyMar 20 2025, 6:21 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM941b4e92c37c: [web] clear old `cryptoStore` before creating new one.

Revision Contents
Changeset List

Path

Size

web/

shared-worker/

worker/

worker-crypto.js

2 lines

Diff 47494

View Options

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

[web] clear old `cryptoStore` before creating new oneClosedPublicActions