Resize listener is referenced to remove it at dispose

[valentinMachado]

Description

In View class the resize listener is referenced, to remove it at dispose

Motivation and Context

This change allow to well dispose a View, and to allow GC to collect View disposed.

[jailln]

Thanks! Does this solves #1936 ? Have you compared memory snapshots before and after removing the view before and after your changes?

[valentinMachado]

Thanks! Does this solves #1936 ? Have you compared memory snapshots before and after removing the view before and after your changes?

I don't know but before this change when a view was disposed the resize listener was still called, meaning the view was still referenced and thus not garbage collected (meaning everything referenced by the view such as layers I guess was not garbage collected, it could solves #1936 or be part of the fix).

In our use case https://github.com/VCityTeam/UD-Imuv we are creating and disposing many PlanarView, what point me this issue was that after some create/dispose a warning was pointing that there were too many webgl context created.

I am still going to investigate this and will check memory snapshot after many create/dispose of a view (VCityTeam/UD-Imuv#410).

[jailln]

It doesn't fixes #1936 but it is still a good fix. Can you rename your commit to follow the angular convention please ? Then we can merge

[valentinMachado]

It doesn't fixes #1936 but it is still a good fix. Can you rename your commit to follow the angular convention please ? Then we can merge

it would have been too simple ^^
done :)