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

Unified Diff: third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js

Issue 2758963002: [DevTools] Remove dependency from TargetManager to ResourceTreeModel (Closed)
Patch Set: review comments addressed Created 3 years, 9 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/network/NetworkLogView.js
diff --git a/third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js b/third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js
index af2120bbea56e3528d83c3a1b589274e1a7d916c..4cef0d366be55bae10ee8ff96e56376d9abfc7d1 100644
--- a/third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js
+++ b/third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js
@@ -741,10 +741,10 @@ Network.NetworkLogView = class extends UI.VBox {
if (!this._recording)
return;
- var data = /** @type {number} */ (event.data);
- if (data) {
- this._mainRequestLoadTime = data;
- this._columns.addEventDividers([data], 'network-red-divider');
+ var time = /** @type {number} */ (event.data.loadTime);
+ if (time) {
+ this._mainRequestLoadTime = time;
+ this._columns.addEventDividers([time], 'network-red-divider');
}
}

Powered by Google App Engine
This is Rietveld 408576698