DevTools: cleanup WI.NetworkMapping nits

DevTools: cleanup WI.NetworkMapping nits

This patch:
- introduces WI.IsolatedFileSystemManager.fileSystems() method. This allows
  to avoid type casts.
- splits out WI.NetworkMapping._addMappingsForFileSystem to avoid creating
  artificial event.

BUG=none
R=dgozman, pfeldman

Committed: https://crrev.com/12918608faeb9c66111e71a4ef1e9393dca5cbe5
Cr-Commit-Position: refs/heads/master@{#415541}

[lushnikov, 2016-08-30 22:54:43]

please, take a look.

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

[lushnikov, 2016-08-30 22:54:49]

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:55:27]

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-31 00:08:35]

https://codereview.chromium.org/2295693003/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/bindings/NetworkMapping.js
(right):

https://codereview.chromium.org/2295693003/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/bindings/NetworkMapping.js:23:
this._addMappingsForFilesystem(fileSystem);
You don't issue fileSystemMappingChanged here anymore.

[lushnikov, 2016-08-31 00:19:30]

https://codereview.chromium.org/2295693003/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/bindings/NetworkMapping.js
(right):

https://codereview.chromium.org/2295693003/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/bindings/NetworkMapping.js:23:
this._addMappingsForFilesystem(fileSystem);
On 2016/08/31 00:08:35, dgozman wrote:
> You don't issue fileSystemMappingChanged here anymore.

it is an event listener

[commit-bot: I haz the power, 2016-08-31 00:44:04]

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

[commit-bot: I haz the power, 2016-08-31 00:44:05]

Dry run: This issue passed the CQ dry run.

[dgozman, 2016-08-31 01:28:34]

lgtm

[lushnikov, 2016-08-31 02:16:06]

The CQ bit was checked by lushnikov@chromium.org

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

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:21:04]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-31 02:24:05]

Description was changed from

==========
DevTools: cleanup WI.NetworkMapping nits

This patch:
- introduces WI.IsolatedFileSystemManager.fileSystems() method. This allows
  to avoid type casts.
- splits out WI.NetworkMapping._addMappingsForFileSystem to avoid creating
  artificial event.

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

to

==========
DevTools: cleanup WI.NetworkMapping nits

This patch:
- introduces WI.IsolatedFileSystemManager.fileSystems() method. This allows
  to avoid type casts.
- splits out WI.NetworkMapping._addMappingsForFileSystem to avoid creating
  artificial event.

BUG=none
R=dgozman, pfeldman

Committed: https://crrev.com/12918608faeb9c66111e71a4ef1e9393dca5cbe5
Cr-Commit-Position: refs/heads/master@{#415541}
==========

[commit-bot: I haz the power, 2016-08-31 02:24:06]

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