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

Unified Diff: third_party/WebKit/Source/devtools/front_end/sdk/NetworkLog.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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/devtools/front_end/sdk/NetworkLog.js
diff --git a/third_party/WebKit/Source/devtools/front_end/sdk/NetworkLog.js b/third_party/WebKit/Source/devtools/front_end/sdk/NetworkLog.js
index 7c0afd8307e6dc21e6597585369ab1fe4dc6c1d3..35c086065c3696fc23a4abba0ea5c094af330a65 100644
--- a/third_party/WebKit/Source/devtools/front_end/sdk/NetworkLog.js
+++ b/third_party/WebKit/Source/devtools/front_end/sdk/NetworkLog.js
@@ -30,29 +30,40 @@
/**
* @constructor
- * @extends {WebInspector.SDKObject}
+ * @extends {WebInspector.SDKModel}
* @param {!WebInspector.Target} target
+ * @param {!WebInspector.NetworkManager} networkManager
*/
-WebInspector.NetworkLog = function(target)
+WebInspector.NetworkLog = function(target, networkManager)
{
- WebInspector.SDKObject.call(this, target);
+ WebInspector.SDKModel.call(this, WebInspector.NetworkLog, target);
this._requests = [];
this._requestForId = {};
- target.networkManager.addEventListener(WebInspector.NetworkManager.EventTypes.RequestStarted, this._onRequestStarted, this);
+ networkManager.addEventListener(WebInspector.NetworkManager.EventTypes.RequestStarted, this._onRequestStarted, this);
target.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.MainFrameNavigated, this._onMainFrameNavigated, this);
target.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.Load, this._onLoad, this);
target.resourceTreeModel.addEventListener(WebInspector.ResourceTreeModel.EventTypes.DOMContentLoaded, this._onDOMContentLoaded, this);
}
/**
+ * @param {!WebInspector.Target} target
+ * @return {?WebInspector.NetworkLog}
+ */
+WebInspector.NetworkLog.fromTarget = function(target)
+{
+ return /** @type {?WebInspector.NetworkLog} */ (target.model(WebInspector.NetworkLog));
+}
+
+/**
* @param {string} url
* @return {?WebInspector.NetworkRequest}
*/
WebInspector.NetworkLog.requestForURL = function(url)
{
for (var target of WebInspector.targetManager.targets()) {
- var result = target.networkLog.requestForURL(url);
+ var networkLog = WebInspector.NetworkLog.fromTarget(target);
+ var result = networkLog && networkLog.requestForURL(url);
if (result)
return result;
}
@@ -66,7 +77,9 @@ WebInspector.NetworkLog.requests = function()
{
var result = [];
for (var target of WebInspector.targetManager.targets()) {
- result = result.concat(target.networkLog.requests());
+ var networkLog = WebInspector.NetworkLog.fromTarget(target);
+ if (networkLog)
+ result = result.concat(networkLog.requests());
}
return result;
}
@@ -162,7 +175,7 @@ WebInspector.NetworkLog.prototype = {
return this._requestForId[requestId];
},
- __proto__: WebInspector.SDKObject.prototype
+ __proto__: WebInspector.SDKModel.prototype
}
/**

Powered by Google App Engine
This is Rietveld 408576698