Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(349)

Side by Side Diff: third_party/WebKit/Source/devtools/front_end/sdk/TargetManager.js

Issue 2109813003: [DevTools] No NetworkManager and NetworkLog for v8only mode (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressing code review comments Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright 2014 The Chromium Authors. All rights reserved. 2 * Copyright 2014 The Chromium Authors. All rights reserved.
3 * Use of this source code is governed by a BSD-style license that can be 3 * Use of this source code is governed by a BSD-style license that can be
4 * found in the LICENSE file. 4 * found in the LICENSE file.
5 */ 5 */
6 6
7 /** 7 /**
8 * @constructor 8 * @constructor
9 * @extends {WebInspector.Object} 9 * @extends {WebInspector.Object}
10 */ 10 */
(...skipping 178 matching lines...) Expand 10 before | Expand all | Expand 10 after
189 * @param {!InspectorBackendClass.Connection} connection 189 * @param {!InspectorBackendClass.Connection} connection
190 * @param {?WebInspector.Target} parentTarget 190 * @param {?WebInspector.Target} parentTarget
191 * @return {!WebInspector.Target} 191 * @return {!WebInspector.Target}
192 */ 192 */
193 createTarget: function(name, type, connection, parentTarget) 193 createTarget: function(name, type, connection, parentTarget)
194 { 194 {
195 var target = new WebInspector.Target(this, name, type, connection, paren tTarget); 195 var target = new WebInspector.Target(this, name, type, connection, paren tTarget);
196 196
197 /** @type {!WebInspector.ConsoleModel} */ 197 /** @type {!WebInspector.ConsoleModel} */
198 target.consoleModel = new WebInspector.ConsoleModel(target); 198 target.consoleModel = new WebInspector.ConsoleModel(target);
199 /** @type {!WebInspector.NetworkManager} */ 199
200 target.networkManager = new WebInspector.NetworkManager(target); 200 var networkManager = null;
201 if (!target.isJSInspector())
202 networkManager = new WebInspector.NetworkManager(target);
203
201 /** @type {!WebInspector.ResourceTreeModel} */ 204 /** @type {!WebInspector.ResourceTreeModel} */
202 target.resourceTreeModel = new WebInspector.ResourceTreeModel(target); 205 target.resourceTreeModel = new WebInspector.ResourceTreeModel(target, ne tworkManager);
203 /** @type {!WebInspector.NetworkLog} */ 206
204 target.networkLog = new WebInspector.NetworkLog(target); 207 if (networkManager)
208 new WebInspector.NetworkLog(target, networkManager);
205 209
206 /** @type {!WebInspector.RuntimeModel} */ 210 /** @type {!WebInspector.RuntimeModel} */
207 target.runtimeModel = new WebInspector.RuntimeModel(target); 211 target.runtimeModel = new WebInspector.RuntimeModel(target);
208 if (target.hasJSContext()) 212 if (target.hasJSContext())
209 new WebInspector.DebuggerModel(target); 213 new WebInspector.DebuggerModel(target);
210 214
211 if (target.type() === WebInspector.Target.Type.Page) { 215 if (target.type() === WebInspector.Target.Type.Page) {
212 new WebInspector.DOMModel(target); 216 new WebInspector.DOMModel(target);
213 new WebInspector.CSSModel(target); 217 new WebInspector.CSSModel(target);
214 } 218 }
215 219
216 /** @type {?WebInspector.WorkerManager} */ 220 /** @type {?WebInspector.WorkerManager} */
217 target.workerManager = !target.isDedicatedWorker() ? new WebInspector.Wo rkerManager(target) : null; 221 target.workerManager = target.supportsWorkers() ? new WebInspector.Worke rManager(target) : null;
218 /** @type {!WebInspector.CPUProfilerModel} */ 222 /** @type {!WebInspector.CPUProfilerModel} */
219 target.cpuProfilerModel = new WebInspector.CPUProfilerModel(target); 223 target.cpuProfilerModel = new WebInspector.CPUProfilerModel(target);
220 /** @type {!WebInspector.HeapProfilerModel} */ 224 /** @type {!WebInspector.HeapProfilerModel} */
221 target.heapProfilerModel = new WebInspector.HeapProfilerModel(target); 225 target.heapProfilerModel = new WebInspector.HeapProfilerModel(target);
222 226
223 target.tracingManager = new WebInspector.TracingManager(target); 227 target.tracingManager = new WebInspector.TracingManager(target);
224 228
225 if (target.isPage()) 229 if (target.isPage())
226 target.serviceWorkerManager = new WebInspector.ServiceWorkerManager( target); 230 target.serviceWorkerManager = new WebInspector.ServiceWorkerManager( target);
227 231
(...skipping 148 matching lines...) Expand 10 before | Expand all | Expand 10 after
376 /** 380 /**
377 * @param {!WebInspector.Target} target 381 * @param {!WebInspector.Target} target
378 */ 382 */
379 targetRemoved: function(target) { }, 383 targetRemoved: function(target) { },
380 } 384 }
381 385
382 /** 386 /**
383 * @type {!WebInspector.TargetManager} 387 * @type {!WebInspector.TargetManager}
384 */ 388 */
385 WebInspector.targetManager = new WebInspector.TargetManager(); 389 WebInspector.targetManager = new WebInspector.TargetManager();
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698