[DevTools] Explicitly require ResourceTreeModel

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 173 matching lines @@
         target.consoleModel = new WebInspector.ConsoleModel(target);
         /** @type {!WebInspector.RuntimeModel} */
         target.runtimeModel = new WebInspector.RuntimeModel(target);
 
         var networkManager = null;
         if (target.hasNetworkCapability())
             networkManager = new WebInspector.NetworkManager(target);
 
         var securityOriginManager = WebInspector.SecurityOriginManager.fromTarget(target);
         /** @type {!WebInspector.ResourceTreeModel} */
         target.resourceTreeModel = new WebInspector.ResourceTreeModel(target, networkManager, securityOriginManager);

[dgozman, 2016/08/17 01:37:52]

Don't assign it here, so that compilation catches more cases.
Can we stop creating it for non-dom targets?

[eostroukhov, 2016/08/17 23:19:38]

I split the CL to make review easier. Next CLs will fix references in other
folders - then we will be able not to create the model.

 
         if (networkManager)
-            new WebInspector.NetworkLog(target, networkManager);
+            new WebInspector.NetworkLog(target, target.resourceTreeModel, networkManager);
 
         if (target.hasJSCapability())
             new WebInspector.DebuggerModel(target);
 
         if (target.hasDOMCapability()) {
-            new WebInspector.DOMModel(target);
-            new WebInspector.CSSModel(target);
+            var domModel = new WebInspector.DOMModel(target);
+            new WebInspector.CSSModel(target, domModel);
         }
 
         /** @type {?WebInspector.WorkerManager} */
         target.workerManager = target.hasWorkerCapability() ? new WebInspector.WorkerManager(target) : null;
         /** @type {!WebInspector.CPUProfilerModel} */
         target.cpuProfilerModel = new WebInspector.CPUProfilerModel(target);
         /** @type {!WebInspector.HeapProfilerModel} */
         target.heapProfilerModel = new WebInspector.HeapProfilerModel(target);
 
         target.tracingManager = new WebInspector.TracingManager(target);
@@ skipping 92 matching lines @@
     },
 
     /**
      * @return {?WebInspector.Target}
      */
     mainTarget: function()
     {
         return this._targets[0] || null;
     },
 
+    /**
+     * @param {!WebInspector.Target} target
+     */
+    suspendReload: function(target)
+    {
+        var resourceTreeModel = WebInspector.ResourceTreeModel.fromTarget(target);
+        if (resourceTreeModel)
+            resourceTreeModel.suspendReload();
+    },
+
+    /**
+     * @param {!WebInspector.Target} target
+     */
+    resumeReload: function(target)
+    {
+        var resourceTreeModel = WebInspector.ResourceTreeModel.fromTarget(target);
+        if (resourceTreeModel)
+            setImmediate(resourceTreeModel.resumeReload.bind(resourceTreeModel));
+    },
+
     __proto__: WebInspector.Object.prototype
 }
 
 /**
  * @interface
  */
 WebInspector.TargetManager.Observer = function()
 {
 }
 
 WebInspector.TargetManager.Observer.prototype = {
     /**
      * @param {!WebInspector.Target} target
      */
     targetAdded: function(target) { },
 
     /**
      * @param {!WebInspector.Target} target
      */
     targetRemoved: function(target) { },
 }
 
 /**
  * @type {!WebInspector.TargetManager}
  */
 WebInspector.targetManager = new WebInspector.TargetManager();