[DevTools] No NetworkManager and NetworkLog for v8only mode

third_party/WebKit/Source/devtools/front_end/sdk/TargetManager.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 {WebInspector.Object}
  */
@@ skipping 178 matching lines @@
      * @param {!InspectorBackendClass.Connection} connection
      * @param {?WebInspector.Target} parentTarget
      * @return {!WebInspector.Target}
      */
     createTarget: function(name, type, connection, parentTarget)
     {
         var target = new WebInspector.Target(this, name, type, connection, parentTarget);
 
         /** @type {!WebInspector.ConsoleModel} */
         target.consoleModel = new WebInspector.ConsoleModel(target);
-        /** @type {!WebInspector.NetworkManager} */
-        target.networkManager = new WebInspector.NetworkManager(target);
+
+        var networkManager = null;
+        if (!target.isV8Inspector()) {

[dgozman, 2016/06/29 18:37:58]

style: extra {}

[eostroukhov-old, 2016/06/29 22:54:21]

Done.

+            networkManager = new WebInspector.NetworkManager(target);
+        }
+
         /** @type {!WebInspector.ResourceTreeModel} */
-        target.resourceTreeModel = new WebInspector.ResourceTreeModel(target);
-        /** @type {!WebInspector.NetworkLog} */
-        target.networkLog = new WebInspector.NetworkLog(target);
+        target.resourceTreeModel = new WebInspector.ResourceTreeModel(target, networkManager);
+
+        if (networkManager) {
+            new WebInspector.NetworkLog(target, networkManager);
+        }
 
         /** @type {!WebInspector.RuntimeModel} */
         target.runtimeModel = new WebInspector.RuntimeModel(target);
         if (target.hasJSContext())
             new WebInspector.DebuggerModel(target);
 
         if (target.type() === WebInspector.Target.Type.Page) {
             new WebInspector.DOMModel(target);
             new WebInspector.CSSModel(target);
         }
 
         /** @type {?WebInspector.WorkerManager} */
-        target.workerManager = !target.isDedicatedWorker() ? new WebInspector.WorkerManager(target) : null;
+        target.workerManager =
+            target.isDedicatedWorker() || target.isV8Inspector() ? null : new WebInspector.WorkerManager(target);

[dgozman, 2016/06/29 18:37:58]

Let's introduce Target.supportsWorkers().

[eostroukhov-old, 2016/06/29 22:54:21]

Done.

         /** @type {!WebInspector.CPUProfilerModel} */
         target.cpuProfilerModel = new WebInspector.CPUProfilerModel(target);
         /** @type {!WebInspector.HeapProfilerModel} */
         target.heapProfilerModel = new WebInspector.HeapProfilerModel(target);
 
         target.tracingManager = new WebInspector.TracingManager(target);
 
         if (target.isPage())
             target.serviceWorkerManager = new WebInspector.ServiceWorkerManager(target);
 
@@ skipping 13 matching lines @@
                 result.push(observer);
         }
         return result;
     },
 
     /**
      * @param {!WebInspector.Target} target
      */
     addTarget: function(target)
     {
+        console.assert(target);

[dgozman, 2016/06/29 18:37:58]

Remove?

[eostroukhov-old, 2016/06/29 22:54:20]

Done.

         this._targets.push(target);
         if (this._targets.length === 1) {
             target.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.InspectedURLChanged, this._redispatchEvent, this);
             target.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.MainFrameNavigated, this._redispatchEvent, this);
             target.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.Load, this._redispatchEvent, this);
             target.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.PageReloadRequested, this._redispatchEvent, this);
             target.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.WillReloadPage, this._redispatchEvent, this);
         }
         var copy = this._observersByType(target.type());
         for (var i = 0; i < copy.length; ++i)
@@ skipping 115 matching lines @@
     /**
      * @param {!WebInspector.Target} target
      */
     targetRemoved: function(target) { },
 }
 
 /**
  * @type {!WebInspector.TargetManager}
  */
 WebInspector.targetManager = new WebInspector.TargetManager();