DevTools: Create target per each worker behind experiment

Source/devtools/front_end/WorkerTargetsProvider.js

+/*
+ * Copyright 2014 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
+ * @param {!WebInspector.Target} mainTarget
+ * @param {!WebInspector.TargetManager} targetManager
+ */
+WebInspector.WorkerTargetsProvider = function(mainTarget, targetManager)

[vsevik, 2014/03/19 15:54:39]

WorkerTargetManager

[sergeyv, 2014/03/19 16:28:59]

Done.

+{
+    this._mainTarget = mainTarget;
+    this._targetManager = targetManager;
+    mainTarget.workerManager.addEventListener(WebInspector.WorkerManager.Events.WorkerAdded, this._onWorkerAdded, this);
+}
+
+WebInspector.WorkerTargetsProvider.prototype = {
+
+    /**
+     * @param {!WebInspector.Event} event
+     */
+    _onWorkerAdded: function(event)
+    {
+        var data = /** @type {{workerId: number, url: string}} */ (event.data);
+        new WebInspector.WorkerConnectionInSameFrontend(this._mainTarget, data.workerId, onConnectionReady.bind(this));

[vsevik, 2014/03/19 15:54:39]

why not var connection = ?

+
+        /**
+         * @this {WebInspector.WorkerTargetsProvider}
+         * @param {!InspectorBackendClass.Connection} connection
+         */
+        function onConnectionReady(connection)

[vsevik, 2014/03/19 15:54:39]

why do you need to pass connection as a callback parameter?

+        {
+            this._targetManager.createTarget(connection, workerTargetInitialization)
+        }
+
+        /**
+         * @param {!WebInspector.Target} target
+         */
+        function workerTargetInitialization(target)
+        {
+            target.runtimeModel.addWorkerContextList(data.url);
+        }
+    }
+
+}
+
+/**
+ * @constructor
+ * @extends {InspectorBackendClass.Connection}
+ * @param {!WebInspector.Target} target
+ * @param {number} workerId
+ * @param {!function(!InspectorBackendClass.Connection)} onConnectionReady
+ */
+WebInspector.WorkerConnectionInSameFrontend = function(target, workerId, onConnectionReady)

[vsevik, 2014/03/19 15:54:39]

Let's name this one WorkerConnection and the other one ExternalWorkerConnection

[sergeyv, 2014/03/19 16:28:59]

Done.

+{
+    InspectorBackendClass.Connection.call(this);
+    this._workerId = workerId;
+    this._workerAgent = target.workerAgent();
+    this._workerAgent.connectToWorker(workerId, onConnectionReady.bind(null, this));
+    target.workerManager.addEventListener(WebInspector.WorkerManager.Events.MessageFromWorker, this._dispatchMessageFromWorker, this);
+}
+
+WebInspector.WorkerConnectionInSameFrontend.prototype = {
+
+    /**
+     * @param {!WebInspector.Event} event
+     */
+    _dispatchMessageFromWorker: function(event)
+    {
+        var data = /** @type {{workerId: number, command: string, message: !Object}} */ (event.data);
+        if (data.workerId === this._workerId)
+            this.dispatch(data.message);
+    },
+
+    /**
+     * @param {!Object} messageObject
+     */
+    sendMessage: function(messageObject)
+    {
+        this._workerAgent.sendMessageToWorker(this._workerId, messageObject);
+    },
+
+    __proto__: InspectorBackendClass.Connection.prototype
+}