DevTools: introduce a stub for the new audits panel (behind experiment).

third_party/WebKit/Source/devtools/front_end/services/ServiceManager.js

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @constructor
  */
 WebInspector.ServiceManager = function()
 {
 }
@@ skipping 12 matching lines @@
                 return /** @type {!Promise<?WebInspector.ServiceManager.Service>} */ (Promise.resolve(null));
             }
             this._remoteConnection = new WebInspector.ServiceManager.Connection(new WebInspector.ServiceManager.RemoteServicePort(url));
         }
         return this._remoteConnection._createService(serviceName);
     },
 
     /**
      * @param {string} appName
      * @param {string} serviceName
-     * @param {boolean} isShared
+     * @param {boolean} isSharedWorker
      * @return {!Promise<?WebInspector.ServiceManager.Service>}
      */
-    createWorkerService: function(appName, serviceName, isShared)
+    createAppService: function(appName, serviceName, isSharedWorker)
     {
-        var connection = new WebInspector.ServiceManager.Connection(new WebInspector.ServiceManager.WorkerServicePort(appName, isShared));
+        var url = appName + ".js";
+        var remoteBase = Runtime.queryParam("remoteBase");
+        if (remoteBase)
+            url += "?remoteBase=" + remoteBase;
+
+        var worker = isSharedWorker ? new SharedWorker(url, appName) : new Worker(url);
+        var connection = new WebInspector.ServiceManager.Connection(new WebInspector.ServiceManager.WorkerServicePort(worker));
         return connection._createService(serviceName);
     }
 }
 
 /**
  * @constructor
  * @param {!ServicePort} port
  */
 WebInspector.ServiceManager.Connection = function(port)
 {
     this._port = port;
     this._port.setHandlers(this._onMessage.bind(this), this._connectionClosed.bind(this));
 
     this._lastId = 1;
     /** @type {!Map<number, function(?Object)>}*/
     this._callbacks = new Map();
     /** @type {!Map<string, !WebInspector.ServiceManager.Service>}*/
     this._services = new Map();
 }
 
 WebInspector.ServiceManager.Connection.prototype = {
     /**
      * @param {string} serviceName
      * @return {!Promise<?WebInspector.ServiceManager.Service>}
      */
     _createService: function(serviceName)
     {
         return this._sendCommand(serviceName + ".create").then(result => {
-            if (!result)
+            if (!result) {
+                console.error("Could not initialize service");

[dgozman, 2016/10/14 03:14:58]

nit: let's print serviceName as well

                 return null;
+            }
             var service = new WebInspector.ServiceManager.Service(this, serviceName, result.id);
             this._services.set(serviceName + ":" + result.id, service);
             return service;
         });
     },
 
     /**
      * @param {!WebInspector.ServiceManager.Service} service
      */
     _serviceDisposed: function(service)
@@ skipping 222 matching lines @@
                 delete this._connectionPromise;
             }
             return true;
         });
     },
 }
 
 /**
  * @constructor
  * @implements {ServicePort}
- * @param {string} appName
- * @param {boolean} isSharedWorker
+ * @param {!Worker|!SharedWorker} worker
  */
-WebInspector.ServiceManager.WorkerServicePort = function(appName, isSharedWorker)
+WebInspector.ServiceManager.WorkerServicePort = function(worker)
 {
-    this._appName = appName;
-    this._isSharedWorker = isSharedWorker;
+    this._worker = worker;
+
+    var callback;

[dgozman, 2016/10/14 03:14:58]

Our pattern has var fulfill, not callback.

+    this._workerPromise = new Promise(fulfill => callback = fulfill);
+    this._isSharedWorker = worker instanceof SharedWorker;
+
+    if (this._isSharedWorker) {
+        this._worker.port.onmessage = onMessage.bind(this);
+        this._worker.port.onclose = this._closeHandler;
+    } else {
+        this._worker.onmessage = onMessage.bind(this);
+        this._worker.onclose = this._closeHandler;
+    }
+
+    /**
+     * @param {!Event} event
+     * @this {WebInspector.ServiceManager.WorkerServicePort}
+     */
+    function onMessage(event)
+    {
+        if (event.data === "workerReady") {
+            callback(true);
+            return;
+        }
+        this._messageHandler(event.data);
+    }
 }
 
 WebInspector.ServiceManager.WorkerServicePort.prototype = {
     /**
      * @override
      * @param {function(string)} messageHandler
      * @param {function(string)} closeHandler
      */
     setHandlers: function(messageHandler, closeHandler)
     {
         this._messageHandler = messageHandler;
         this._closeHandler = closeHandler;
     },
 
     /**
-     * @return {!Promise<boolean>}
-     */
-    _open: function()
-    {
-        if (this._workerPromise)
-            return this._workerPromise;
-
-        var url = this._appName + ".js";
-        var remoteBase = Runtime.queryParam("remoteBase");
-        if (remoteBase)
-            url += "?remoteBase=" + remoteBase;
-
-        this._workerPromise = new Promise(promiseBody.bind(this));
-
-        /**
-         * @param {function(boolean)} fulfill
-         * @this {WebInspector.ServiceManager.WorkerServicePort}
-         */
-        function promiseBody(fulfill)
-        {
-            if (this._isSharedWorker) {
-                this._worker = new SharedWorker(url, this._appName);
-                this._worker.port.onmessage = onMessage.bind(this);
-                this._worker.port.onclose = this._closeHandler;
-            } else {
-                this._worker = new Worker(url);
-                this._worker.onmessage = onMessage.bind(this);
-                this._worker.onclose = this._closeHandler;
-            }
-
-            /**
-             * @param {!Event} event
-             * @this {WebInspector.ServiceManager.WorkerServicePort}
-             */
-            function onMessage(event)
-            {
-                if (event.data === "workerReady") {
-                    fulfill(true);
-                    return;
-                }
-                this._messageHandler(event.data);
-            }
-        }
-        return this._workerPromise;
-    },
-
-    /**
      * @override
      * @param {string} message
      * @return {!Promise<boolean>}
      */
     send: function(message)
     {
-        return this._open().then(() => {
+        return this._workerPromise.then(() => {
             try {
                 if (this._isSharedWorker)
                     this._worker.port.postMessage(message);
                 else
                     this._worker.postMessage(message);
                 return true;
             } catch (e) {
                 return false;
             }
         });
     },
 
     /**
      * @override
      * @return {!Promise}
      */
     close: function()
     {
-        return this._open().then(() => {
+        return this._workerPromise.then(() => {
             if (this._worker)
                 this._worker.terminate();
             return false;
         });
     }
 }
 
 WebInspector.serviceManager = new WebInspector.ServiceManager();