DevTools: do not leak memory through event listeners

DevTools: do not leak memory through event listeners

EventListeners array hold references to the objects which
they were listening to, preventing them from being
cleaned up.

BUG=none
R=dgozman

Committed: https://crrev.com/86b94ffee79cb6ad57adcdbf6af8dea839a9b729
Cr-Commit-Position: refs/heads/master@{#415545}

[lushnikov, 2016-08-30 21:42:28]

Dmitry, this is a follow-up to our offline discussion.

In fact, I think that we do not leak memory anywhere, and this is not required.
Let me know what you think

[dgozman, 2016-08-30 23:00:07]

I like this patch! I think it will save us later. lgtm

[lushnikov, 2016-08-31 00:52:02]

Description was changed from

==========
[DO NOT LAND] DevTools: do not leak memory through event listeners

BUG=none
R=dgozman
==========

to

==========
DevTools: do not leak memory through event listeners

EventListeners array hold references to the objects which
they were listening to, preventing them from being
cleaned up.

BUG=none
R=dgozman
==========

[lushnikov, 2016-08-31 00:52:07]

The CQ bit was checked by lushnikov@chromium.org

[lushnikov, 2016-08-31 00:52:07]

The patchset sent to the CQ was uploaded after l-g-t-m from dgozman@chromium.org
Link to the patchset: https://codereview.chromium.org/2298473003/#ps20001
(title: "cleanup - some clients have being cleaning event listeners")

[commit-bot: I haz the power, 2016-08-31 00:52:26]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-31 02:47:39]

Description was changed from

==========
DevTools: do not leak memory through event listeners

EventListeners array hold references to the objects which
they were listening to, preventing them from being
cleaned up.

BUG=none
R=dgozman
==========

to

==========
DevTools: do not leak memory through event listeners

EventListeners array hold references to the objects which
they were listening to, preventing them from being
cleaned up.

BUG=none
R=dgozman
==========

[commit-bot: I haz the power, 2016-08-31 02:47:40]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-08-31 02:49:16]

Description was changed from

==========
DevTools: do not leak memory through event listeners

EventListeners array hold references to the objects which
they were listening to, preventing them from being
cleaned up.

BUG=none
R=dgozman
==========

to

==========
DevTools: do not leak memory through event listeners

EventListeners array hold references to the objects which
they were listening to, preventing them from being
cleaned up.

BUG=none
R=dgozman

Committed: https://crrev.com/86b94ffee79cb6ad57adcdbf6af8dea839a9b729
Cr-Commit-Position: refs/heads/master@{#415545}
==========

[commit-bot: I haz the power, 2016-08-31 02:49:17]

Patchset 2 (id:??) landed as
https://crrev.com/86b94ffee79cb6ad57adcdbf6af8dea839a9b729
Cr-Commit-Position: refs/heads/master@{#415545}