[web] free memory in `createAndPersistNotificationsOutboundSession`
ClosedPublic
Actions

Authored by kamil on Feb 11 2025, 2:24 PM.

Details

Reviewers

tomek
bartek

Commits

rCOMM64a7e1117b29: [web] free memory in `createAndPersistNotificationsOutboundSession`

Summary

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

Tracked in ENG-10183.

Freeing memory after the session is persisted.

Depends on D14351

Test Plan

Test notifs on web

Diff Detail

Repository

rCOMM Comm

Branch

memory-leaks-fixes-web

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

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

kamil held this revision as a draft.

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

kamil added a child revision: D14353: [web] fix memory management in `getOrCreateOlmAccount`.Feb 11 2025, 2:35 PM

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

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

Harbormaster completed remote builds in B33470: Diff 47003.Feb 11 2025, 3:06 PM

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

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

Closed by commit rCOMM64a7e1117b29: [web] free memory in `createAndPersistNotificationsOutboundSession`. · Explain WhyFeb 12 2025, 4:48 PM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM64a7e1117b29: [web] free memory in `createAndPersistNotificationsOutboundSession`.

Revision Contents
Changeset List

Path

Size

web/

shared-worker/

worker/

worker-crypto.js

3 lines

Diff 47003

View Options

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

[web] free memory in `createAndPersistNotificationsOutboundSession`ClosedPublicActions