[DevTools]: Require explicit connection

third_party/WebKit/Source/devtools/front_end/sdk/SubTargetsManager.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.
 /**
  * @unrestricted
  */
 WebInspector.SubTargetsManager = class extends WebInspector.SDKModel {
   /**
    * @param {!WebInspector.Target} target
    */
   constructor(target) {
     super(WebInspector.SubTargetsManager, target);
     target.registerTargetDispatcher(new WebInspector.SubTargetsDispatcher(this));
     this._lastAnonymousTargetId = 0;
     this._agent = target.targetAgent();
 
     /** @type {!Map<string, !WebInspector.Target>} */
     this._attachedTargets = new Map();
     /** @type {!Map<string, !WebInspector.SubTargetConnection>} */
     this._connections = new Map();
-
+    /** @type {!Map<string, !WebInspector.PendingTarget>} */
+    this._pendingTargets = new Map();
     this._agent.setAutoAttach(true /* autoAttach */, true /* waitForDebuggerOnStart */);
     if (Runtime.experiments.isEnabled('autoAttachToCrossProcessSubframes'))
       this._agent.setAttachToFrames(true);
 
     if (Runtime.experiments.isEnabled('nodeDebugging') && !target.parentTarget()) {
       var defaultLocations = [{host: 'localhost', port: 9229}];
       this._agent.setRemoteLocations(defaultLocations);
       this._agent.setDiscoverTargets(true);
     }
     WebInspector.targetManager.addEventListener(
         WebInspector.TargetManager.Events.MainFrameNavigated, this._mainFrameNavigated, this);
   }
 
   /**
+   * @param {!WebInspector.Target} target
+   * @return {?WebInspector.SubTargetsManager}
+   */
+  static fromTarget(target)
+  {
+    return /** @type {?WebInspector.SubTargetsManager} */ (target.model(WebInspector.SubTargetsManager));
+  }
+
+  /**
    * @override
    * @return {!Promise}
    */
   suspendModel() {
     var fulfill;
     var promise = new Promise(f => fulfill = f);
     this._agent.setAutoAttach(true /* autoAttach */, false /* waitForDebuggerOnStart */, fulfill);
     return promise;
   }
 
   /**
    * @override
    * @return {!Promise}
    */
   resumeModel() {
     var fulfill;
     var promise = new Promise(f => fulfill = f);
     this._agent.setAutoAttach(true /* autoAttach */, true /* waitForDebuggerOnStart */, fulfill);
     return promise;
   }
 
   /**
    * @override
    */
   dispose() {
-    for (var connection of this._connections.values()) {
-      this._agent.detachFromTarget(connection._targetId);
-      connection._onDisconnect.call(null, 'disposed');
-    }
-    this._connections.clear();
-    this._attachedTargets.clear();
+    for (var attachedTargetId of this._attachedTargets.keys())
+      this._detachedFromTarget(attachedTargetId);
+    for (var pendingConnectionId of this._pendingTargets.keys())
+      this._targetDestroyed(pendingConnectionId);
   }
 
   /**
    * @param {!Protocol.Target.TargetID} targetId
    */
   activateTarget(targetId) {
     this._agent.activateTarget(targetId);
   }
 
   /**
@@ skipping 72 matching lines @@
     target[WebInspector.SubTargetsManager._InfoSymbol] = targetInfo;
     this._attachedTargets.set(targetInfo.id, target);
 
     // Only pause new worker if debugging SW - we are going through the pause on start checkbox.
     var mainIsServiceWorker =
         !this.target().parentTarget() && this.target().hasTargetCapability() && !this.target().hasBrowserCapability();
     if (mainIsServiceWorker && waitingForDebugger)
       target.debuggerAgent().pause();
     target.runtimeAgent().runIfWaitingForDebugger();
 
-    this.dispatchEventToListeners(WebInspector.SubTargetsManager.Events.SubTargetAdded, target);
+    var pendingTarget = this._pendingTargets.get(targetInfo.id);
+    if (!pendingTarget) {
+      this._targetCreated(targetInfo);
+      pendingTarget = this._pendingTargets.get(targetInfo.id);
+    }
+    pendingTarget.notifyAttached(target);
+    this.dispatchEventToListeners(WebInspector.SubTargetsManager.Events.SubTargetAttached, pendingTarget);
   }
 
   /**
    * @param {string} targetId
    * @param {!InspectorBackendClass.Connection.Params} params
    * @return {!InspectorBackendClass.Connection}
    */
   _createConnection(targetId, params) {
     var connection = new WebInspector.SubTargetConnection(this._agent, targetId, params);
     this._connections.set(targetId, connection);
     return connection;
   }
 
   /**
    * @param {string} targetId
    */
   _detachedFromTarget(targetId) {
     var target = this._attachedTargets.get(targetId);
     this._attachedTargets.delete(targetId);
-    this.dispatchEventToListeners(WebInspector.SubTargetsManager.Events.SubTargetRemoved, target);
     var connection = this._connections.get(targetId);
-    if (connection)
-      connection._onDisconnect.call(null, 'target terminated');
+    connection._onDisconnect.call(null, 'target terminated');
     this._connections.delete(targetId);
+    this.dispatchEventToListeners(WebInspector.SubTargetsManager.Events.SubTargetDetached, this._pendingTargets.get(targetId));
   }
 
   /**
    * @param {string} targetId
    * @param {string} message
    */
   _receivedMessageFromTarget(targetId, message) {
     var connection = this._connections.get(targetId);
     if (connection)
       connection._onMessage.call(null, message);
   }
 
   /**
    * @param {!WebInspector.TargetInfo} targetInfo
    */
   _targetCreated(targetInfo) {
-    if (targetInfo.type !== 'node')
+    var pendingTarget = this._pendingTargets.get(targetInfo.id);
+    if (pendingTarget)
       return;
-    this._agent.attachToTarget(targetInfo.id);
+    pendingTarget = new WebInspector.PendingTarget(targetInfo.id, targetInfo.title, targetInfo.type === 'node', this);
+    this._pendingTargets.set(targetInfo.id, pendingTarget);
+    this.dispatchEventToListeners(WebInspector.SubTargetsManager.Events.SubTargetAdded, pendingTarget);
   }
 
   /**
    * @param {string} targetId
    */
   _targetDestroyed(targetId) {
-    // All the work is done in _detachedFromTarget.
+    var pendingTarget = this._pendingTargets.get(targetId);
+    if (!pendingTarget)
+      return;
+    this._pendingTargets.delete(targetId);
+    this.dispatchEventToListeners(WebInspector.SubTargetsManager.Events.SubTargetRemoved, pendingTarget);
   }
 
   /**
+   * @return {!Array<!WebInspector.PendingTarget>}
+   */
+  pendingTargets() {
+    return this._pendingTargets.valuesArray();
+  }
+
+  /**
    * @param {!WebInspector.Event} event
    */
   _mainFrameNavigated(event) {
     if (event.data.target() !== this.target())
       return;
 
     var idsToDetach = [];
     for (var targetId of this._attachedTargets.keys()) {
       var target = this._attachedTargets.get(targetId);
       var targetInfo = this.targetInfo(target);
       if (targetInfo.type === 'worker')
         idsToDetach.push(targetId);
     }
     idsToDetach.forEach(id => this._detachedFromTarget(id));
   }
 };
 
 /** @enum {symbol} */
 WebInspector.SubTargetsManager.Events = {
   SubTargetAdded: Symbol('SubTargetAdded'),

[dgozman, 2016/11/03 21:21:30]

PendingTarget{Added,Removed,Attached,Detached}

[eostroukhov, 2016/11/03 22:44:50]

Done.

   SubTargetRemoved: Symbol('SubTargetRemoved'),
+  SubTargetAttached: Symbol('SubTargetAttached'),
+  SubTargetDetached: Symbol('SubTargetDetached'),
 };
 
 WebInspector.SubTargetsManager._InfoSymbol = Symbol('SubTargetInfo');
 
 /**
  * @implements {Protocol.TargetDispatcher}
  * @unrestricted
  */
 WebInspector.SubTargetsDispatcher = class {
   /**
@@ skipping 93 matching lines @@
     this.type = payload.type;
     this.canActivate = this.type === 'page' || this.type === 'iframe';
     if (this.type === 'node')
       this.title = WebInspector.UIString('Node: %s', this.url);
     else if (this.type === 'page' || this.type === 'iframe')
       this.title = payload.title;
     else
       this.title = WebInspector.UIString('Worker: %s', this.url);
   }
 };
+
+/**
+ * @unrestricted
+ */
+WebInspector.PendingTarget = class {
+  /**
+   * @param {string} id
+   * @param {string} title
+   * @param {boolean} canConnect
+   * @param {?WebInspector.SubTargetsManager} manager
+   */
+  constructor(id, title, canConnect, manager) {
+    this._id = id;
+    this._title = title;
+    this._isRemote = canConnect;
+    this._manager = manager;
+    /** @type {?Promise} */
+    this._connectPromise = null;
+    /** @type {?Function} */
+    this._attachedCallback = null;
+  }
+
+  /**
+   * @return {string}
+   */
+  id() {
+    return this._id;
+  }
+
+  /**
+   * @return {?WebInspector.Target}
+   */
+  target() {
+    if (!this._manager)
+      return null;
+    return this._manager.targetForId(this.id());
+  }
+
+  /**
+   * @return {string}
+   */
+  name()
+  {

[dgozman, 2016/11/03 21:21:30]

{ on the same line everywhere please.

[eostroukhov, 2016/11/03 22:44:50]

Done.

+    return this._title;
+  }
+
+  /**
+   * @return {!Promise}
+   */
+  connect()

[dgozman, 2016/11/03 21:21:30]

Let's call these attach/detach to be consistent.

[eostroukhov, 2016/11/03 22:44:50]

Done.

+  {
+    if (!this._manager)
+      return Promise.reject();
+    if (this._connectPromise)
+      return this._connectPromise;
+    if (this.target())
+      return Promise.resolve(this.target());
+    this._connectPromise = new Promise(resolve => {
+      this._attachedCallback = resolve;
+      this._manager._agent.attachToTarget(this.id());
+    });
+    return this._connectPromise;
+  }
+
+  disconnect()
+  {
+    if (!this._manager)
+      return;
+    this._manager._agent.detachFromTarget(this.id());
+  }
+
+  /**
+   * @param {!WebInspector.Target} target
+   */
+  notifyAttached(target)
+  {
+    if (this._attachedCallback)
+      this._attachedCallback(target);
+    this._connectPromise = null;
+    this._attachedCallback = null;
+  }
+
+  /**
+   * @return {boolean}
+   */
+  canConnect()
+  {
+    return this._isRemote;
+  }
+};