[DevTools] Replace the target type with capabilities

third_party/WebKit/Source/devtools/front_end/sdk/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 {!WebInspector.TargetManager} targetManager
  * @param {string} name
- * @param {number} type
+ * @param {number} capabilitiesMask
  * @param {!InspectorBackendClass.Connection} connection
  * @param {?WebInspector.Target} parentTarget
  */
-WebInspector.Target = function(targetManager, name, type, connection, parentTarget)
+WebInspector.Target = function(targetManager, name, capabilitiesMask, connection, parentTarget)
 {
     Protocol.Agents.call(this, connection.agentsMap());
     this._targetManager = targetManager;
     this._name = name;
-    this._type = type;
+    this._capabilitiesMask = capabilitiesMask;
     this._connection = connection;
     this._parentTarget = parentTarget;
     connection.addEventListener(InspectorBackendClass.Connection.Events.Disconnected, this._onDisconnect, this);
     this._id = WebInspector.Target._nextId++;
 
     /** @type {!Map.<!Function, !WebInspector.SDKModel>} */
     this._modelByConstructor = new Map();
 }
 
 /**
  * @enum {number}
  */
-WebInspector.Target.Type = {
-    Page: 1,
-    DedicatedWorker: 2,
-    ServiceWorker: 4,
-    JSInspector: 8
+WebInspector.Target.Capabilities = {

[dgozman, 2016/07/12 00:34:11]

Capability

[eostroukhov-old, 2016/07/12 21:46:16]

Done.

+    BROWSER_DOMAINS: 1,

[dgozman, 2016/07/12 00:34:11]

Browser, JS, Network, Worker

[eostroukhov-old, 2016/07/12 21:46:15]

Done.

+    JS_DOMAINS: 2,
+    NETWORK_DOMAINS: 4,
+    WORKER_DOMAINS: 8,
 }
 
 WebInspector.Target._nextId = 1;
 
 WebInspector.Target.prototype = {
     /**
      * @return {number}
      */
     id: function()
     {
         return this._id;
     },
 
     /**
      * @return {string}
      */
     name: function()
     {
         return this._name;
     },
 
     /**
-     *
-     * @return {number}
-     */
-    type: function()
-    {
-        return this._type;
-    },
-
-    /**
-     *
      * @return {!WebInspector.TargetManager}
      */
     targetManager: function()
     {
         return this._targetManager;
     },
 
     /**
+     * @return {number}
+     */
+    capabilitiesMask: function()

[dgozman, 2016/07/12 00:34:11]

Who uses this? Let's not expose.

[eostroukhov-old, 2016/07/12 21:46:16]

It was used by the TargetManager - now that have changed.

+    {
+        return this._capabilitiesMask;
+    },
+
+    /**
      * @param {string} label
      * @return {string}
      */
     decorateLabel: function(label)
     {
-        return this.isWorker() ? "\u2699 " + label : label;
+        return !this.hasBrowserDomains() ? "\u2699 " + label : label;
     },
 
     /**
      * @override
      * @param {string} domain
      * @param {!Object} dispatcher
      */
     registerDispatcher: function(domain, dispatcher)
     {
         this._connection.registerDispatcher(domain, dispatcher);
     },
 
+    _hasCapability: function(capability)

[dgozman, 2016/07/12 00:34:11]

Annotate please.

[eostroukhov-old, 2016/07/12 21:46:16]

This function is now gone.

+    {
+        return (this._capabilitiesMask & capability) === capability;
+    },
+
     /**
      * @return {boolean}
      */
-    isPage: function()
+    hasBrowserDomains: function()

[dgozman, 2016/07/12 00:34:11]

Now I think maybe we should call this hasBrowserCapability. WDYT?

[eostroukhov-old, 2016/07/12 21:46:16]

Done.

     {
-        return this._type === WebInspector.Target.Type.Page;
+        return this._hasCapability(WebInspector.Target.Capabilities.BROWSER_DOMAINS);
     },
 
     /**
      * @return {boolean}
      */
-    isWorker: function()
+    hasJSDomains: function()
     {
-        return this.isDedicatedWorker() || this.isServiceWorker() || this.isJSInspector();
+        return this._hasCapability(WebInspector.Target.Capabilities.JS_DOMAINS);
     },
 
     /**
      * @return {boolean}
      */
-    isDedicatedWorker: function()
+    hasNetworkDomains: function()
     {
-        return this._type === WebInspector.Target.Type.DedicatedWorker;
+        return this._hasCapability(WebInspector.Target.Capabilities.NETWORK_DOMAINS);
     },
 
     /**
      * @return {boolean}
      */
-    isServiceWorker: function()
+    hasWorkerDomains: function()
     {
-        return this._type === WebInspector.Target.Type.ServiceWorker;
+        return this._hasCapability(WebInspector.Target.Capabilities.WORKER_DOMAINS);
     },
 
     /**
-     * @return {boolean}
-     */
-    isJSInspector: function()
-    {
-        return this._type === WebInspector.Target.Type.JSInspector;
-    },
-
-    /**
-     * @return {boolean}
-     */
-    hasJSContext: function()
-    {
-        return !this.isServiceWorker();
-    },
-
-    /**
-     * @return {boolean}
-     */
-    supportsWorkers: function()
-    {
-        return this.isPage() || this.isServiceWorker();
-    },
-
-    /**
      * @return {?WebInspector.Target}
      */
     parentTarget: function()
     {
         return this._parentTarget;
     },
 
     _onDisconnect: function()
     {
         this._targetManager.removeTarget(this);
@@ skipping 97 matching lines @@
     {
         var target = /** @type {!WebInspector.Target} */ (event.data);
         if (target !== this._target)
             return;
         this.dispose();
         WebInspector.targetManager.removeEventListener(WebInspector.TargetManager.Events.TargetDisposed, this._targetDisposed, this);
     },
 
     __proto__: WebInspector.SDKObject.prototype
 }