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

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

Issue 2431223003: [DevTools]: Require explicit connection (Closed)
Patch Set: Reworked - again Created 4 years, 1 month 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 167 matching lines...) Expand 10 before | Expand all | Expand 10 after
178 { 178 {
179 delete targetObserver[this._observerCapabiliesMaskSymbol]; 179 delete targetObserver[this._observerCapabiliesMaskSymbol];
180 this._observers.remove(targetObserver); 180 this._observers.remove(targetObserver);
181 }, 181 },
182 182
183 /** 183 /**
184 * @param {string} name 184 * @param {string} name
185 * @param {number} capabilitiesMask 185 * @param {number} capabilitiesMask
186 * @param {!InspectorBackendClass.Connection.Factory} connectionFactory 186 * @param {!InspectorBackendClass.Connection.Factory} connectionFactory
187 * @param {?WebInspector.Target} parentTarget 187 * @param {?WebInspector.Target} parentTarget
188 * @param {boolean=} createOnly
188 * @return {!WebInspector.Target} 189 * @return {!WebInspector.Target}
189 */ 190 */
190 createTarget: function(name, capabilitiesMask, connectionFactory, parentTarg et) 191 createTarget: function(name, capabilitiesMask, connectionFactory, parentTarg et, createOnly)
191 { 192 {
192 var target = new WebInspector.Target(this, name, capabilitiesMask, conne ctionFactory, parentTarget); 193 var target = new WebInspector.Target(this, name, capabilitiesMask, conne ctionFactory, parentTarget);
193 194
194 var logAgent = target.hasLogCapability() ? target.logAgent() : null; 195 var logAgent = target.hasLogCapability() ? target.logAgent() : null;
195 196
196 /** @type {!WebInspector.ConsoleModel} */ 197 /** @type {!WebInspector.ConsoleModel} */
197 target.consoleModel = new WebInspector.ConsoleModel(target, logAgent); 198 target.consoleModel = new WebInspector.ConsoleModel(target, logAgent);
198 199
199 var networkManager = null; 200 var networkManager = null;
200 var resourceTreeModel = null; 201 var resourceTreeModel = null;
(...skipping 21 matching lines...) Expand all
222 /** @type {!WebInspector.CPUProfilerModel} */ 223 /** @type {!WebInspector.CPUProfilerModel} */
223 target.cpuProfilerModel = new WebInspector.CPUProfilerModel(target); 224 target.cpuProfilerModel = new WebInspector.CPUProfilerModel(target);
224 /** @type {!WebInspector.HeapProfilerModel} */ 225 /** @type {!WebInspector.HeapProfilerModel} */
225 target.heapProfilerModel = new WebInspector.HeapProfilerModel(target); 226 target.heapProfilerModel = new WebInspector.HeapProfilerModel(target);
226 227
227 target.tracingManager = new WebInspector.TracingManager(target); 228 target.tracingManager = new WebInspector.TracingManager(target);
228 229
229 if (target.subTargetsManager && target.hasBrowserCapability()) 230 if (target.subTargetsManager && target.hasBrowserCapability())
230 target.serviceWorkerManager = new WebInspector.ServiceWorkerManager( target, target.subTargetsManager); 231 target.serviceWorkerManager = new WebInspector.ServiceWorkerManager( target, target.subTargetsManager);
231 232
232 this.addTarget(target); 233 if (!createOnly)
234 this.addTarget(target);
233 return target; 235 return target;
234 }, 236 },
235 237
236 /** 238 /**
237 * @param {function(function(string)):!Promise<!InspectorBackendClass.Connec tion>} interceptor 239 * @param {function(function(string)):!Promise<!InspectorBackendClass.Connec tion>} interceptor
238 */ 240 */
239 setMainConnectionInterceptor: function(interceptor) 241 setMainConnectionInterceptor: function(interceptor)
240 { 242 {
241 this._mainConnectionInterceptor = interceptor; 243 this._mainConnectionInterceptor = interceptor;
242 }, 244 },
(...skipping 156 matching lines...) Expand 10 before | Expand all | Expand 10 after
399 /** 401 /**
400 * @param {!WebInspector.Target} target 402 * @param {!WebInspector.Target} target
401 */ 403 */
402 targetRemoved: function(target) { }, 404 targetRemoved: function(target) { },
403 }; 405 };
404 406
405 /** 407 /**
406 * @type {!WebInspector.TargetManager} 408 * @type {!WebInspector.TargetManager}
407 */ 409 */
408 WebInspector.targetManager = new WebInspector.TargetManager(); 410 WebInspector.targetManager = new WebInspector.TargetManager();
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698