angular-ui/ui-grid

headerCanvas, headerViewPort Memory Leak

Open

#3.889 geöffnet am 29. Juni 2015

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (2.496 Forks)batch import
help wantedseverity: memory leakseverity: performancetype: bug

Repository-Metriken

Stars
 (5.395 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

I am running into a huge memory leak problem, where memory usage increase by 50m+ every time I switch between tabs in a single page application.

Grids are dynamically generated depending on the configuration of tab objects pulled asynchronously from a restful service. Each tab can contain up to four grids.

I followed the "three snapshot" technique and narrowed the problem down to thousands of RED nodes under headerCanvas and headerViewPort objects under containerCtrl.

memory_leak

ENV

AngularJS v1.3.16
UI-Grid 3.0.0-rc.21
Memory leak confirmed on IE11, Firefox, and Chrome

I have tried upgrading to RC 22 and upgraded Angular to 1.3.16. The problem still persist.

Contributor Guide

Research-Richtung
Erstellen Sie Speicher Snapshots in Chrome DevTools, um beibehaltene Knoten unter headerCanvas und headerViewPort zu identifizieren; überprüfen Sie, ob Ereignis Listener oder Scope Watcher beim Wechseln von Tabs nicht bereinigt werden.
Tech Stack
javascript
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
AngularJSbrowser debugging tools
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60