DevTools: Introduce Target class which holds connection & models

Source/devtools/front_end/Target.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
+ * @extends {Protocol.Agents}
+ * @param {!InspectorBackendClass.Connection} connection
+ * @param {function(!WebInspector.Target)=} callback
+ */
+WebInspector.Target = function(connection, callback)
+{
+    Protocol.Agents.call(this, connection.agentsMap());
+    this._connection = connection;
+    this.canScreenCast = false;

[vsevik, 2014/03/06 17:52:31]

Why is this needed?

[sergeyv, 2014/03/07 09:42:08]

Done.

+    this.canInspectWorkers = false;
+
+    this.pageAgent().canScreencast(this._initializeCapability.bind(this, "canScreencast", null));
+    this.workerAgent().canInspectWorkers(this._initializeCapability.bind(this, "canInspectWorkers", this._loadedWithCapabilities.bind(this, callback)));
+
+    // In case of loading as a web page with no bindings / harness, kick off initialization manually.
+    if (InspectorFrontendHost.isStub) {
+        // give a chance to add listeners of WebInspector.Target.Events.TargetIsReady
+        setTimeout(this._loadedWithCapabilities.bind(this, callback), 0);
+    }
+}
+
+WebInspector.Target.prototype = {
+
+     _initializeCapability: function(name, callback, error, result)

[vsevik, 2014/03/06 17:52:31]

weird indent

[sergeyv, 2014/03/07 09:42:08]

Done.

+    {
+        this[name] = result;
+        if (callback)
+            callback();
+    },
+
+    /**
+     * @param {function(!WebInspector.Target)=} callback
+     */
+    _loadedWithCapabilities: function(callback)
+    {
+        // compatibility code
+        Capabilities.canInspectWorkers = this.canInspectWorkers;

[vsevik, 2014/03/06 17:52:31]

Capabilities[name] = result; in _initializeCapability instead, let's this legacy
code shorter and in one place.

[sergeyv, 2014/03/07 09:42:08]

Done.

+
+        this.consoleModel = new WebInspector.ConsoleModel();
+        //This and similar lines are needed for compatibility
+        WebInspector.console = this.consoleModel;
+        this.networkManager = new WebInspector.NetworkManager();
+        WebInspector.networkManager = this.networkManager;
+        this.resourceTreeModel = new WebInspector.ResourceTreeModel(this.networkManager);
+        WebInspector.resourceTreeModel = this.resourceTreeModel;
+        this.debuggerModel = new WebInspector.DebuggerModel();
+        WebInspector.debuggerModel = this.debuggerModel;
+        this.runtimeModel = new WebInspector.RuntimeModel(this.resourceTreeModel);
+        WebInspector.runtimeModel = this.runtimeModel;
+
+        //we can't name it domAgent, because it clashes with function, WebInspector.DOMAgent should be renamed to DOMModel
+        this._domAgent = new WebInspector.DOMAgent();
+        WebInspector.domAgent = this._domAgent;
+        this.workerManager = new WebInspector.WorkerManager(this.canInspectWorkers);
+        WebInspector.workerManager = this.workerManager;
+
+        if (callback)
+            callback(this);
+    },
+
+    __proto__: Protocol.Agents.prototype
+}
+
+/**
+ * @constructor
+ */
+WebInspector.TargetManager = function()
+{
+    /** @type {!Array.<!WebInspector.Target>} */
+    this._targets = [];
+}
+
+WebInspector.TargetManager.prototype = {
+
+    /**
+     * @param {!InspectorBackendClass.Connection} connection
+     * @param {function(!WebInspector.Target)=} callback
+     */
+    createTarget: function(connection, callback)
+    {
+        var newTarget = new WebInspector.Target(connection, callback);
+        this._targets.push(newTarget);
+    }
+
+}