cssinjs/jss

Global SheetsRegistry leaks memory?

Open

#1340 aperta il 3 mag 2020

Vedi su GitHub
 (0 commenti) (1 reazione) (0 assegnatari)JavaScript (434 fork)batch import
help wantedperf

Metriche repository

Star
 (7000 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

Expected behavior: Created StyleSheet instances should not be retained indefinitely.

Describe the bug: StyleSheet instances are being retained inside the global SheetsRegistry.

Codesandbox link: Run the react-jss example code https://github.com/cssinjs/react-todomvc-jss.git

  1. Take memory snapshot.
  2. Add a few items.
  3. Delete those items.
  4. Repeat step 2 and 3 a few more times.
  5. Take another memory snapshot.
  6. Filter both snapshots by constructor named StyleSheet and you'll find the number only grows.

Versions (please complete the following information):

  • jss: The example is using 6.5.0, but I believe the issue is present in all versions.
  • Browser [e.g. chrome, safari]: Chrome
  • OS [e.g. Windows, macOS]: MacOS Feel free to add any additional versions which you may think are relevant to the bug.

Guida contributor

Direzione di ricerca
Esamina il SheetsRegistry globale in JSS per capire perché le istanze di StyleSheet vengono trattenute. Controlla se manca una pulizia o un riferimento che impedisce la garbage collection dopo lo smontaggio dei componenti. Studia il ciclo di vita di attach/detach dei fogli di stile.
Tech stack
javascript
Dominio
frontend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitBasic understanding of JavaScript and JSS
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60