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

third_party/WebKit/Source/devtools/front_end/bindings/FileSystemWorkspaceBinding.js

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 19 matching lines @@
 
 /**
  * @constructor
  * @param {!WebInspector.IsolatedFileSystemManager} isolatedFileSystemManager
  * @param {!WebInspector.Workspace} workspace
  */
 WebInspector.FileSystemWorkspaceBinding = function(isolatedFileSystemManager, workspace)
 {
     this._isolatedFileSystemManager = isolatedFileSystemManager;
     this._workspace = workspace;
-    this._isolatedFileSystemManager.addEventListener(WebInspector.IsolatedFileSystemManager.Events.FileSystemAdded, this._fileSystemAdded, this);
-    this._isolatedFileSystemManager.addEventListener(WebInspector.IsolatedFileSystemManager.Events.FileSystemRemoved, this._fileSystemRemoved, this);
-    this._isolatedFileSystemManager.addEventListener(WebInspector.IsolatedFileSystemManager.Events.FileSystemFilesChanged, this._fileSystemFilesChanged, this);
+    this._eventListeners = [
+        this._isolatedFileSystemManager.addEventListener(WebInspector.IsolatedFileSystemManager.Events.FileSystemAdded, this._fileSystemAdded, this),
+        this._isolatedFileSystemManager.addEventListener(WebInspector.IsolatedFileSystemManager.Events.FileSystemRemoved, this._fileSystemRemoved, this),
+        this._isolatedFileSystemManager.addEventListener(WebInspector.IsolatedFileSystemManager.Events.FileSystemFilesChanged, this._fileSystemFilesChanged, this)
+    ];
     /** @type {!Map.<string, !WebInspector.FileSystemWorkspaceBinding.FileSystem>} */
     this._boundFileSystems = new Map();
 }
 
 WebInspector.FileSystemWorkspaceBinding._styleSheetExtensions = new Set(["css", "scss", "sass", "less"]);
 WebInspector.FileSystemWorkspaceBinding._documentExtensions = new Set(["htm", "html", "asp", "aspx", "phtml", "jsp"]);
 WebInspector.FileSystemWorkspaceBinding._scriptExtensions = new Set(["asp", "aspx", "c", "cc", "cljs", "coffee", "cpp", "cs", "dart", "java", "js", "jsp", "jsx", "h", "m", "mm", "py", "sh", "ts", "tsx", "ls"]);
 
 WebInspector.FileSystemWorkspaceBinding._imageExtensions = WebInspector.IsolatedFileSystem.ImageExtensions;
 
@@ skipping 93 matching lines @@
      * @param {string} projectId
      * @return {string}
      */
     fileSystemPath: function(projectId)
     {
         return projectId;
     },
 
     dispose: function()
     {
-        this._isolatedFileSystemManager.removeEventListener(WebInspector.IsolatedFileSystemManager.Events.FileSystemAdded, this._fileSystemAdded, this);
-        this._isolatedFileSystemManager.removeEventListener(WebInspector.IsolatedFileSystemManager.Events.FileSystemRemoved, this._fileSystemRemoved, this);
-        this._isolatedFileSystemManager.dispose();
+        WebInspector.EventTarget.removeEventListeners(this._eventListeners);
         for (var fileSystem of this._boundFileSystems.values()) {
             fileSystem.dispose();
             this._boundFileSystems.remove(fileSystem._fileSystem.path());
         }
     }
 }
 
 /**
  * @constructor
  * @extends {WebInspector.ProjectStore}
@@ skipping 346 matching lines @@
         uiSourceCode.checkContentUpdated();
     },
 
     dispose: function()
     {
         this.removeProject();
     },
 
     __proto__: WebInspector.ProjectStore.prototype
 }