DevTools: use eventListener arrays to avoid memory leaks in dispose methods

DevTools: use eventListener arrays to avoid memory leaks in dispose methods

This patch:

- switches to using event listener arrays in FileSystemWorkspaceBinding
  and NetworkMapping classes.
- removes IsolatedFileSystemManager.dispose method. IFSM is a singleton
  which is supposed to be always alive.

BUG=none
R=dgozman, pfeldman

Committed: https://crrev.com/9e0ff32923aa6f94f4f53cf412888afec5138e71
Cr-Commit-Position: refs/heads/master@{#415494}

[lushnikov, 2016-08-30 21:59:27]

please, take a look!

FYI: this is a part of crrev.com/2241573002

[lushnikov, 2016-08-30 21:59:54]

The CQ bit was checked by lushnikov@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-30 22:00:39]

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

[dgozman, 2016-08-30 23:02:32]

lgtm

[commit-bot: I haz the power, 2016-08-30 23:30:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-30 23:30:36]

Dry run: This issue passed the CQ dry run.

[lushnikov, 2016-08-30 23:32:53]

The CQ bit was checked by lushnikov@chromium.org

[commit-bot: I haz the power, 2016-08-30 23:33:50]

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-30 23:38:44]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-30 23:40:12]

Description was changed from

==========
DevTools: use eventListener arrays to avoid memory leaks in dispose methods

This patch:

- switches to using event listener arrays in FileSystemWorkspaceBinding
  and NetworkMapping classes.
- removes IsolatedFileSystemManager.dispose method. IFSM is a singleton
  which is supposed to be always alive.

BUG=none
R=dgozman, pfeldman
==========

to

==========
DevTools: use eventListener arrays to avoid memory leaks in dispose methods

This patch:

- switches to using event listener arrays in FileSystemWorkspaceBinding
  and NetworkMapping classes.
- removes IsolatedFileSystemManager.dispose method. IFSM is a singleton
  which is supposed to be always alive.

BUG=none
R=dgozman, pfeldman

Committed: https://crrev.com/9e0ff32923aa6f94f4f53cf412888afec5138e71
Cr-Commit-Position: refs/heads/master@{#415494}
==========

[commit-bot: I haz the power, 2016-08-30 23:40:13]

Patchset 1 (id:??) landed as
https://crrev.com/9e0ff32923aa6f94f4f53cf412888afec5138e71
Cr-Commit-Position: refs/heads/master@{#415494}