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

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

Issue 2168323002: [DevTools] Explicitly require ResourceTreeModel (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/devtools/front_end/sdk/TargetManager.js
diff --git a/third_party/WebKit/Source/devtools/front_end/sdk/TargetManager.js b/third_party/WebKit/Source/devtools/front_end/sdk/TargetManager.js
index ca96560e22f8d254eb50b16a0721e281b9222b92..b9f4fe63787534ad1f6356b1d6cbb092f4080b4b 100644
--- a/third_party/WebKit/Source/devtools/front_end/sdk/TargetManager.js
+++ b/third_party/WebKit/Source/devtools/front_end/sdk/TargetManager.js
@@ -207,14 +207,14 @@ WebInspector.TargetManager.prototype = {
target.resourceTreeModel = new WebInspector.ResourceTreeModel(target, networkManager);
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, target.resourceTreeModel);
}
/** @type {?WebInspector.WorkerManager} */
@@ -226,7 +226,7 @@ WebInspector.TargetManager.prototype = {
target.tracingManager = new WebInspector.TracingManager(target);
- if (target.hasBrowserCapability())
+ if (target.hasDOMCapability())
dgozman 2016/07/22 17:19:44 Looks strange. How DOM capability is related to se
eostroukhov-old 2016/07/25 19:36:11 Done.
target.serviceWorkerManager = new WebInspector.ServiceWorkerManager(target);
this.addTarget(target);

Powered by Google App Engine
This is Rietveld 408576698