angular-ui/ui-grid

headerCanvas, headerViewPort Memory Leak

Open

#3.889 aberto em 29 de jun. de 2015

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)JavaScript (2.496 forks)batch import
help wantedseverity: memory leakseverity: performancetype: bug

Métricas do repositório

Stars
 (5.395 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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.

Guia do colaborador

Direção de pesquisa
Analise os snapshots de memória no Chrome DevTools para identificar nós retidos sob headerCanvas e headerViewPort; verifique se há ouvintes de eventos ou observadores de escopo que não foram limpos ao alternar entre guias.
Pilha de tecnologia
javascript
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Principalmente claro
Prerequisites
AngularJSbrowser debugging tools
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60