DevTools: kill WI.IsolatedFileSystemManager FileSystemsLoaded event

third_party/WebKit/Source/devtools/front_end/workspace/IsolatedFileSystemManager.js

 /*
  * Copyright (C) 2012 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 33 matching lines @@
     InspectorFrontendHost.events.addEventListener(InspectorFrontendHostAPI.Events.FileSystemsLoaded, this._onFileSystemsLoaded, this);
     InspectorFrontendHost.events.addEventListener(InspectorFrontendHostAPI.Events.FileSystemRemoved, this._onFileSystemRemoved, this);
     InspectorFrontendHost.events.addEventListener(InspectorFrontendHostAPI.Events.FileSystemAdded, this._onFileSystemAdded, this);
     InspectorFrontendHost.events.addEventListener(InspectorFrontendHostAPI.Events.FileSystemFilesChanged, this._onFileSystemFilesChanged, this);
     InspectorFrontendHost.events.addEventListener(InspectorFrontendHostAPI.Events.IndexingTotalWorkCalculated, this._onIndexingTotalWorkCalculated, this);
     InspectorFrontendHost.events.addEventListener(InspectorFrontendHostAPI.Events.IndexingWorked, this._onIndexingWorked, this);
     InspectorFrontendHost.events.addEventListener(InspectorFrontendHostAPI.Events.IndexingDone, this._onIndexingDone, this);
     InspectorFrontendHost.events.addEventListener(InspectorFrontendHostAPI.Events.SearchCompleted, this._onSearchCompleted, this);
 
     this._initExcludePatterSetting();
+
+    this._fileSystemsLoadedPromise = new Promise(fulfill => this._fileSystemsLoadedCallback = fulfill);

[dgozman, 2016/08/31 01:39:46]

this._fileSystemsLoadedPromise = this._requestFileSystems();



_requestFileSystems: function()
{
    var fulfill;
    var promise = new Promise(f => fulfill = f);
    InspectorFrontendHost.requestFileSystems();
   
InspectorFrontendHost.events.addEventListener(InspectorFrontendHostAPI.Events.FileSystemsLoaded,
onFileSystemsLoaded, this);
    return promise;

    function onFileSystemsLoaded()
    {
        ...
        fulfill();
    }
}

[lushnikov, 2016/08/31 17:35:10]

Done.

+    InspectorFrontendHost.requestFileSystems();
 }
 
 /** @typedef {!{fileSystemName: string, rootURL: string, fileSystemPath: string}} */
 WebInspector.IsolatedFileSystemManager.FileSystem;
 
 /** @enum {symbol} */
 WebInspector.IsolatedFileSystemManager.Events = {
     FileSystemAdded: Symbol("FileSystemAdded"),
     FileSystemRemoved: Symbol("FileSystemRemoved"),
-    FileSystemsLoaded: Symbol("FileSystemsLoaded"),
     FileSystemFilesChanged: Symbol("FileSystemFilesChanged"),
     ExcludedFolderAdded: Symbol("ExcludedFolderAdded"),
     ExcludedFolderRemoved: Symbol("ExcludedFolderRemoved")
 }
 
 WebInspector.IsolatedFileSystemManager._lastRequestId = 0;
 
 /**
  * @param {string} fileSystemPath
  * @return {string}
  */
 WebInspector.IsolatedFileSystemManager.normalizePath = function(fileSystemPath)
 {
     fileSystemPath = fileSystemPath.replace(/\\/g, "/");
     if (!fileSystemPath.startsWith("file://")) {
         if (fileSystemPath.startsWith("/"))
             fileSystemPath = "file://" + fileSystemPath;
         else
             fileSystemPath = "file:///" + fileSystemPath;
     }
     return fileSystemPath;
 }
 
 WebInspector.IsolatedFileSystemManager.prototype = {
-    /**
-     * @param {function()} callback
-     */
-    initialize: function(callback)
-    {
-        this._initializeCallback = callback;
-        InspectorFrontendHost.requestFileSystems();
-    },
-
     addFileSystem: function()
     {
         InspectorFrontendHost.addFileSystem("");
     },
 
     /**
      * @param {!WebInspector.IsolatedFileSystem} fileSystem
      */
     removeFileSystem: function(fileSystem)
     {
-
         InspectorFrontendHost.removeFileSystem(fileSystem.embedderPath());
     },
 
     /**
      * @param {!WebInspector.Event} event
      */
     _onFileSystemsLoaded: function(event)
     {
         var fileSystems = /** @type {!Array.<!WebInspector.IsolatedFileSystemManager.FileSystem>} */ (event.data);
         var promises = [];
         for (var i = 0; i < fileSystems.length; ++i)
-            promises.push(this._innerAddFileSystem(fileSystems[i]));
-        Promise.all(promises).then(fireFileSystemsLoaded.bind(this));
-
-        /**
-         * @this {WebInspector.IsolatedFileSystemManager}
-         */
-        function fireFileSystemsLoaded()
-        {
-            this._initializeCallback();
-            delete this._initializeCallback;
-            this.dispatchEventToListeners(WebInspector.IsolatedFileSystemManager.Events.FileSystemsLoaded);
-        }
+            promises.push(this._innerAddFileSystem(fileSystems[i], false));
+        Promise.all(promises).then(this._fileSystemsLoadedCallback);
     },
 
     /**
-     * @return {boolean}
+     * @return {!Promise}
      */
-    fileSystemsLoaded: function()
+    waitForFileSystems: function()
     {
-        return !this._initializeCallback;
+        return this._fileSystemsLoadedPromise;
     },
 
     /**
      * @param {!WebInspector.IsolatedFileSystemManager.FileSystem} fileSystem
+     * @param {boolean} dispatchEvent
      * @return {!Promise}
      */
-    _innerAddFileSystem: function(fileSystem)
+    _innerAddFileSystem: function(fileSystem, dispatchEvent)
     {
         var embedderPath = fileSystem.fileSystemPath;
         var fileSystemPath = WebInspector.IsolatedFileSystemManager.normalizePath(fileSystem.fileSystemPath);
         var promise = WebInspector.IsolatedFileSystem.create(this, fileSystemPath, embedderPath, fileSystem.fileSystemName, fileSystem.rootURL);
         return promise.then(storeFileSystem.bind(this));
 
         /**
          * @param {?WebInspector.IsolatedFileSystem} fileSystem
          * @this {WebInspector.IsolatedFileSystemManager}
          */
         function storeFileSystem(fileSystem)
         {
             if (!fileSystem)
                 return;
             this._fileSystems.set(fileSystemPath, fileSystem);
-            this.dispatchEventToListeners(WebInspector.IsolatedFileSystemManager.Events.FileSystemAdded, fileSystem);
+            if (dispatchEvent)
+                this.dispatchEventToListeners(WebInspector.IsolatedFileSystemManager.Events.FileSystemAdded, fileSystem)

[dgozman, 2016/08/31 01:39:46]

semicolon

[lushnikov, 2016/08/31 17:35:10]

Done.

         }
     },
 
     /**
      * @param {!WebInspector.Event} event
      */
     _onFileSystemAdded: function(event)
     {
         var errorMessage = /** @type {string} */ (event.data["errorMessage"]);
         var fileSystem = /** @type {?WebInspector.IsolatedFileSystemManager.FileSystem} */ (event.data["fileSystem"]);
         if (errorMessage)
             WebInspector.console.error(errorMessage);
         else if (fileSystem)
-            this._innerAddFileSystem(fileSystem);
+            this._innerAddFileSystem(fileSystem, true)

[dgozman, 2016/08/31 01:39:46]

semicolon

[lushnikov, 2016/08/31 17:35:10]

Done.

     },
 
     /**
      * @param {!WebInspector.Event} event
      */
     _onFileSystemRemoved: function(event)
     {
-        this._fileSystemRemoved(/** @type {string} */ (event.data));
+        var embedderPath = /** @type {string} */ (event.data);

[lushnikov, 2016/08/31 00:26:58]

this is just inlined _fileSystemRemoved method.

+        var fileSystemPath = WebInspector.IsolatedFileSystemManager.normalizePath(embedderPath);
+        var isolatedFileSystem = this._fileSystems.get(fileSystemPath);
+        if (!isolatedFileSystem)
+            return;
+        this._fileSystems.delete(fileSystemPath);
+        isolatedFileSystem.fileSystemRemoved();
+        this.dispatchEventToListeners(WebInspector.IsolatedFileSystemManager.Events.FileSystemRemoved, isolatedFileSystem);
     },
 
     /**
      * @param {!WebInspector.Event} event
      */
     _onFileSystemFilesChanged: function(event)
     {
         var embedderPaths = /** @type {!Array<string>} */ (event.data);
         var paths = embedderPaths.map(embedderPath => WebInspector.IsolatedFileSystemManager.normalizePath(embedderPath));
         this.dispatchEventToListeners(WebInspector.IsolatedFileSystemManager.Events.FileSystemFilesChanged, paths);
     },
 
     /**
-     * @param {string} embedderPath
-     */
-    _fileSystemRemoved: function(embedderPath)
-    {
-        var fileSystemPath = WebInspector.IsolatedFileSystemManager.normalizePath(embedderPath);
-        var isolatedFileSystem = this._fileSystems.get(fileSystemPath);
-        if (!isolatedFileSystem)
-            return;
-        this._fileSystems.delete(fileSystemPath);
-        isolatedFileSystem.fileSystemRemoved();
-        this.dispatchEventToListeners(WebInspector.IsolatedFileSystemManager.Events.FileSystemRemoved, isolatedFileSystem);
-    },
-
-    /**
      * @return {!Array<!WebInspector.IsolatedFileSystem>}
      */
     fileSystems: function()
     {
         return this._fileSystems.valuesArray();
     },
 
     /**
      * @param {string} fileSystemPath
      * @return {?WebInspector.IsolatedFileSystem}
@@ skipping 135 matching lines @@
         this._callbacks.delete(requestId);
     },
 
     __proto__: WebInspector.Object.prototype
 }
 
 /**
  * @type {!WebInspector.IsolatedFileSystemManager}
  */
 WebInspector.isolatedFileSystemManager;