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

Authored by kamil on Thu, Mar 20, 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.Thu, Mar 20, 4:59 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptThu, Mar 20, 4:59 AM

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

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

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

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

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

Closed by commit rCOMM941b4e92c37c: [web] clear old `cryptoStore` before creating new one (authored by kamil). · Explain WhyThu, Mar 20, 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