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

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

Issue 2436953003: [Devtools] Highlight navigation request in network timeline exp (Closed)
Patch Set: [Devtools] Highlight navigation request in network timeline exp Created 4 years, 2 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 034b627fc202e7bdc8b1f7aa066e1a235373b462..b28588abcf63cc527f6c750c551adac8630b329c 100644
--- a/third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js
+++ b/third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js
@@ -329,8 +329,20 @@ WebInspector.NetworkLogView.prototype = {
_redrawTimelineColumn: function()
{
- if (this._timelineRequestsAreStale)
- this._timelineColumn.setRequests(this._getOrderedRequests());
+ if (!this._timelineRequestsAreStale) {
+ this._timelineColumn.draw();
+ return;
+ }
+
+ var currentNode = this._dataGrid.rootNode();
+ var requestData = [];
+ var navigationRequest = null;
+ while (currentNode = currentNode.traverseNextNode(true)) {
+ if (currentNode.isNavigationRequest())
+ navigationRequest = currentNode.request();
+ requestData.push(currentNode.request());
+ }
+ this._timelineColumn.setRequests(requestData, navigationRequest);
this._timelineColumn.draw();
},
@@ -729,15 +741,6 @@ WebInspector.NetworkLogView.prototype = {
this._timelineRequestsAreStale = true;
},
- _getOrderedRequests: function()
- {
- var currentNode = this._dataGrid.rootNode();
- var requestData = [];
- while (currentNode = currentNode.traverseNextNode(true))
- requestData.push(currentNode.request());
- return requestData;
- },
-
reset: function()
{
this._requestWithHighlightedInitiators = null;

Powered by Google App Engine
This is Rietveld 408576698