[Devtools] Prepare network log view for grouping support

third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js

 /*
  * Copyright (C) 2007, 2008 Apple Inc.  All rights reserved.
  * Copyright (C) 2008, 2009 Anthony Ricaud <rik@webkit.org>
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
@@ skipping 487 matching lines @@
    * @return {!Array.<!Element>}
    */
   elementsToRestoreScrollPositionsFor() {
     if (!this._dataGrid)  // Not initialized yet.
       return [];
     return [this._dataGrid.scrollContainer];
   }
 
   _setupDataGrid() {
     this._dataGrid = this._columns.dataGrid();
+    var contextMenuHandler = this.handleContextMenuForLogEntry.bind(this);
     this._dataGrid.setRowContextMenuCallback(
-        (contextMenu, node) => this.handleContextMenuForRequest(contextMenu, node.request()));
+        (contextMenu, node) => contextMenuHandler(contextMenu, /** @type {!Network.NetworkDataGridNode} */ (node)));
     this._dataGrid.setStickToBottom(true);
     this._dataGrid.setName('networkLog');
     this._dataGrid.setResizeMethod(UI.DataGrid.ResizeMethod.Last);
     this._dataGrid.element.classList.add('network-log-grid');
     this._dataGrid.element.addEventListener('mousedown', this._dataGridMouseDown.bind(this), true);
     this._dataGrid.element.addEventListener('mousemove', this._dataGridMouseMove.bind(this), true);
     this._dataGrid.element.addEventListener('mouseleave', this._dataGridMouseLeave.bind(this), true);
   }
 
   /**
    * @param {!Event} event
    */
   _dataGridMouseMove(event) {
     var node = this._dataGrid.dataGridNodeFromNode(event.target);
     var highlightInitiatorChain = event.shiftKey;
     this._setHoveredNode(node, highlightInitiatorChain);
-    this._highlightInitiatorChain((highlightInitiatorChain && node) ? node.request() : null);
+    // TODO(allada) Support groupping initiator chain instead of first request.
+    var firstRequest = node ? node.firstRequest() : null;
+    this._highlightInitiatorChain(highlightInitiatorChain ? firstRequest : null);
   }
 
   _dataGridMouseLeave() {
     this._setHoveredNode(null);
     this._highlightInitiatorChain(null);
   }
 
   /**
    * @param {?Network.NetworkLogEntry} logEntry
    * @param {boolean} highlightInitiatorChain
    */
   setHoveredLogEntry(logEntry, highlightInitiatorChain) {
     // TODO(allada) Move this into LogEntry/NetworkDataGridNode.
     this._setHoveredNode(/** @type {?Network.NetworkDataGridNode} */ (logEntry), highlightInitiatorChain);
-    this._highlightInitiatorChain((logEntry && highlightInitiatorChain) ? logEntry.request() : null);
+    // TODO(allada) Support groupping initiator chain instead of first request.
+    this._highlightInitiatorChain((logEntry && highlightInitiatorChain) ? logEntry.firstRequest() : null);
   }
 
   /**
    * @param {?Network.NetworkDataGridNode} node
    * @param {boolean=} highlightInitiatorChain
    */
   _setHoveredNode(node, highlightInitiatorChain) {
     if (this._hoveredNode)
       this._hoveredNode.element().classList.remove('hover');
     this._hoveredNode = node;
@@ skipping 24 matching lines @@
           continue;
         node.element().classList.remove('network-node-on-initiator-path', 'network-node-on-initiated-path');
       }
       return;
     }
 
     var initiatorGraph = request.initiatorGraph();
     for (var node of this._nodesByRequestId.values()) {
       if (!node.dataGrid)
         continue;
-      node.element().classList.toggle(
-          'network-node-on-initiator-path',
-          node.request() !== request && initiatorGraph.initiators.has(node.request()));
-      node.element().classList.toggle(
-          'network-node-on-initiated-path', node.request() !== request && initiatorGraph.initiated.has(node.request()));
+      var hasInitiator = false;
+      var hasInitiated = false;
+      for (var childRequest of node.requests()) {
+        if (childRequest === request)
+          continue;
+        hasInitiator = initiatorGraph.initiators.has(childRequest);
+        hasInitiated = initiatorGraph.initiated.has(childRequest);
+        if (hasInitiator || hasInitiated)
+          break;
+      }
+
+      node.element().classList.toggle('network-node-on-initiator-path', hasInitiator);
+      node.element().classList.toggle('network-node-on-initiated-path', hasInitiated);
     }
   }
 
   _updateSummaryBar() {
     var requestsNumber = this._nodesByRequestId.size;
 
     if (!requestsNumber) {
       this._showRecordingHint();
       return;
     }
     this._hideRecordingHint();
 
     var transferSize = 0;
     var selectedRequestsNumber = 0;
     var selectedTransferSize = 0;
     var baseTime = -1;
     var maxTime = -1;
-    var nodes = this._nodesByRequestId.valuesArray();
-    for (var i = 0; i < nodes.length; ++i) {
-      var request = nodes[i].request();
-      var requestTransferSize = request.transferSize;
-      transferSize += requestTransferSize;
-      if (!nodes[i][Network.NetworkLogView._isFilteredOutSymbol]) {
-        selectedRequestsNumber++;
-        selectedTransferSize += requestTransferSize;
+    for (var node of this._nodesByRequestId.values()) {
+      var isFilteredOut = node[Network.NetworkLogView._isFilteredOutSymbol];
+      for (var request of node.requests()) {
+        var requestTransferSize = request.transferSize;
+        transferSize += requestTransferSize;
+        if (!isFilteredOut) {
+          selectedRequestsNumber++;
+          selectedTransferSize += requestTransferSize;
+        }
+        if (request.url === request.target().inspectedURL() && request.resourceType() === Common.resourceTypes.Document)
+          baseTime = request.startTime;
+        if (request.endTime > maxTime)
+          maxTime = request.endTime;
       }
-      if (request.url === request.target().inspectedURL() && request.resourceType() === Common.resourceTypes.Document)
-        baseTime = request.startTime;
-      if (request.endTime > maxTime)
-        maxTime = request.endTime;
     }
 
     var summaryBar = this._summaryBarElement;
     summaryBar.removeChildren();
     var separator = '\u2002\u2758\u2002';
     var text = '';
     /**
      * @param {string} chunk
      * @return {!Element}
      */
@@ skipping 167 matching lines @@
     this.removeAllNodeHighlights();
 
     var oldBoundary = this.calculator().boundary();
     this._timeCalculator.updateBoundariesForEventTime(this._mainRequestLoadTime);
     this._durationCalculator.updateBoundariesForEventTime(this._mainRequestLoadTime);
     this._timeCalculator.updateBoundariesForEventTime(this._mainRequestDOMContentLoadedTime);
     this._durationCalculator.updateBoundariesForEventTime(this._mainRequestDOMContentLoadedTime);
 
     var dataGrid = this._dataGrid;
     var rootNode = dataGrid.rootNode();
+
     /** @type {!Array<!Network.NetworkDataGridNode> } */
     var nodesToInsert = [];
     /** @type {!Array<!Network.NetworkDataGridNode> } */
     var nodesToRefresh = [];
     for (var requestId in this._staleRequestIds) {
       var node = this._nodesByRequestId.get(requestId);
       if (!node)
         continue;
       var isFilteredOut = !this._applyFilter(node);
       if (isFilteredOut && node === this._hoveredNode)
         this._setHoveredNode(null);
       if (node[Network.NetworkLogView._isFilteredOutSymbol] !== isFilteredOut) {
         if (!node[Network.NetworkLogView._isFilteredOutSymbol])
           rootNode.removeChild(node);
 
         node[Network.NetworkLogView._isFilteredOutSymbol] = isFilteredOut;
 
         if (!node[Network.NetworkLogView._isFilteredOutSymbol])
           nodesToInsert.push(node);
       }
       if (!isFilteredOut)
         nodesToRefresh.push(node);
-      var request = node.request();
-      this._timeCalculator.updateBoundaries(request);
-      this._durationCalculator.updateBoundaries(request);
+      for (var request of node.requests()) {
+        this._timeCalculator.updateBoundaries(request);
+        this._durationCalculator.updateBoundaries(request);
+      }
     }
 
-    for (var i = 0; i < nodesToInsert.length; ++i) {
-      var node = nodesToInsert[i];
-      var request = node.request();
-      dataGrid.insertChild(node);
-      node[Network.NetworkLogView._isMatchingSearchQuerySymbol] = this._matchRequest(request);
+    for (var node of nodesToInsert) {
+      for (var request of node.requests()) {
+        dataGrid.insertChild(node);

[dgozman, 2016/11/23 01:57:30]

This should go outside of the for loop.

[allada, 2016/11/23 22:39:14]

Done.

+        node[Network.NetworkLogView._isMatchingSearchQuerySymbol] = this._matchRequest(request);
+      }
     }
 
     for (var node of nodesToRefresh)
       node.refresh();
 
     this._highlightNthMatchedRequestForSearch(
         this._updateMatchCountAndFindMatchIndex(this._currentMatchedRequestNode), false);
 
     this._staleRequestIds = {};
     this._updateSummaryBar();
@@ skipping 56 matching lines @@
    * @param {!SDK.NetworkRequest} request
    */
   _appendRequest(request) {
     var node = new Network.NetworkDataGridNode(this, request);
     node[Network.NetworkLogView._isFilteredOutSymbol] = true;
     node[Network.NetworkLogView._isMatchingSearchQuerySymbol] = false;
 
     // In case of redirect request id is reassigned to a redirected
     // request and we need to update _nodesByRequestId and search results.
     var originalRequestNode = this._nodesByRequestId.get(request.requestId);
-    if (originalRequestNode)
-      this._nodesByRequestId.set(originalRequestNode.request().requestId, originalRequestNode);
+    if (originalRequestNode) {
+      for (var originalRequest of originalRequestNode.requests())
+        this._nodesByRequestId.set(originalRequest.requestId, originalRequestNode);

[dgozman, 2016/11/23 01:57:30]

I believe this is incorrect now - only a single requestid should be reassigned.

[allada, 2016/11/23 22:39:14]

Done.

+    }
+
     this._nodesByRequestId.set(request.requestId, node);
 
     // Pull all the redirects of the main request upon commit load.
-    if (request.redirects) {
-      for (var i = 0; i < request.redirects.length; ++i)
-        this._refreshRequest(request.redirects[i]);
-    }
+    if (request.redirects)
+      request.redirects.forEach(this._refreshRequest.bind(this));
 
     this._refreshRequest(request);
   }
 
   /**
    * @param {!Common.Event} event
    */
   _onRequestUpdated(event) {
     var request = /** @type {!SDK.NetworkRequest} */ (event.data);
     this._refreshRequest(request);
@@ skipping 90 matching lines @@
 
   /**
    * @param {boolean} gridMode
    */
   switchViewMode(gridMode) {
     this._columns.switchViewMode(gridMode);
   }
 
   /**
    * @param {!UI.ContextMenu} contextMenu
-   * @param {!SDK.NetworkRequest} request
+   * @param {!Network.NetworkLogEntry} logEntry
    */
-  handleContextMenuForRequest(contextMenu, request) {
+  handleContextMenuForLogEntry(contextMenu, logEntry) {
+    // TODO(allada) Support groupped items context menu.
+    var request = logEntry.firstRequest();
+    if (logEntry.isGroup() || !request)
+      return;
     contextMenu.appendApplicableItems(request);
     var copyMenu = contextMenu.appendSubMenuItem(Common.UIString('Copy'));
     if (request) {
       copyMenu.appendItem(
           UI.copyLinkAddressLabel(), InspectorFrontendHost.copyText.bind(InspectorFrontendHost, request.contentURL()));
       copyMenu.appendSeparator();
 
       if (request.requestHeadersText()) {
         copyMenu.appendItem(
             Common.UIString.capitalize('Copy ^request ^headers'), this._copyRequestHeaders.bind(this, request));
@@ skipping 49 matching lines @@
     }
 
     if (request && request.resourceType() === Common.resourceTypes.XHR) {
       contextMenu.appendSeparator();
       contextMenu.appendItem(Common.UIString('Replay XHR'), request.replayXHR.bind(request));
       contextMenu.appendSeparator();
     }
   }
 
   _harRequests() {
-    var requests = this._nodesByRequestId.valuesArray().map(function(node) {
-      return node.request();
-    });
+    var requests = [];
+    for (var node of this._nodesByRequestId.values())
+      requests = requests.concat(node.requests());
     var httpRequests = requests.filter(Network.NetworkLogView.HTTPRequestsFilter);
     return httpRequests.filter(Network.NetworkLogView.FinishedRequestsFilter);
   }
 
   _copyAll() {
     var harArchive = {log: (new SDK.HARLog(this._harRequests())).build()};
     InspectorFrontendHost.copyText(JSON.stringify(harArchive, null, 2));
   }
 
   /**
@@ skipping 30 matching lines @@
    * @param {string} platform
    */
   _copyCurlCommand(request, platform) {
     InspectorFrontendHost.copyText(this._generateCurlCommand(request, platform));
   }
 
   /**
    * @param {string} platform
    */
   _copyAllCurlCommand(platform) {
-    var requests = this._nodesByRequestId.valuesArray().map(node => node.request());
+    var requests = [];
+    for (var node of this._nodesByRequestId.values())

[dgozman, 2016/11/23 01:57:30]

Let's have a helper method which does this.

[allada, 2016/11/23 22:39:14]

Done.

+      requests = requests.concat(node.requests());
     var commands = [];
     for (var request of requests)
       commands.push(this._generateCurlCommand(request, platform));
     if (platform === 'win')
       InspectorFrontendHost.copyText(commands.join(' &\r\n'));
     else
       InspectorFrontendHost.copyText(commands.join(' ;\n'));
   }
 
   _exportAll() {
@@ skipping 58 matching lines @@
         this._updateMatchCountAndFindMatchIndex(this._currentMatchedRequestNode), false);
   }
 
   /**
    * @param {number} n
    * @param {boolean} reveal
    */
   _highlightNthMatchedRequestForSearch(n, reveal) {
     this._removeAllHighlights();
 
-    /** @type {!Array.<!Network.NetworkDataGridNode>} */
-    var nodes = this._dataGrid.rootNode().children;
     var matchCount = 0;
     var node = null;
-    for (var i = 0; i < nodes.length; ++i) {
-      if (nodes[i][Network.NetworkLogView._isMatchingSearchQuerySymbol]) {
+    var request = null;
+    for (node of this.flatNodesList()) {

[dgozman, 2016/11/23 01:57:30]

var node

[allada, 2016/11/23 22:39:14]

Done.

+      if (node[Network.NetworkLogView._isMatchingSearchQuerySymbol]) {
         if (matchCount === n) {
-          node = nodes[i];
+          // TODO(allada) This should support multiple requests.
+          request = node.firstRequest();
           break;
         }
         matchCount++;
       }
     }
-    if (!node) {
+    if (!request) {
       this._currentMatchedRequestNode = null;
       return;
     }
 
-    var request = node.request();
     if (reveal)
       Common.Revealer.reveal(request);
     var highlightedSubstringChanges = node.highlightMatchedSubstring(this._searchRegex);
     this._highlightedSubstringChanges.push(highlightedSubstringChanges);
 
     this._currentMatchedRequestNode = node;
     this._currentMatchedRequestIndex = n;
     this.dispatchEventToListeners(Network.NetworkLogView.Events.SearchIndexUpdated, n);
   }
 
   /**
    * @override
    * @param {!UI.SearchableView.SearchConfig} searchConfig
    * @param {boolean} shouldJump
    * @param {boolean=} jumpBackwards
    */
   performSearch(searchConfig, shouldJump, jumpBackwards) {
     var query = searchConfig.query;
     var currentMatchedRequestNode = this._currentMatchedRequestNode;
     this._clearSearchMatchedList();
     this._searchRegex = createPlainTextSearchRegex(query, 'i');
 
-    /** @type {!Array.<!Network.NetworkDataGridNode>} */
-    var nodes = this._dataGrid.rootNode().children;
-    for (var i = 0; i < nodes.length; ++i)
-      nodes[i][Network.NetworkLogView._isMatchingSearchQuerySymbol] = this._matchRequest(nodes[i].request());
+    var isMatchingSearchQuerySymbol = Network.NetworkLogView._isMatchingSearchQuerySymbol;

[dgozman, 2016/11/23 01:57:30]

Inline it back :-)

[allada, 2016/11/23 22:39:14]

Done.

+    for (var node of this.flatNodesList()) {

[dgozman, 2016/11/23 01:57:30]

Is this really an equivalent of what was here before? I think one is "visible"
while the other is "all".

[allada, 2016/11/23 22:39:14]

Done.

+      var firstRequest = node.firstRequest();
+      // TODO(allada) This should properly support groupped requests.

[dgozman, 2016/11/23 01:57:30]

definitely!

[allada, 2016/11/23 22:39:14]

Acknowledged.

+      node[isMatchingSearchQuerySymbol] = firstRequest ? this._matchRequest(firstRequest) : null;
+    }
     var newMatchedRequestIndex = this._updateMatchCountAndFindMatchIndex(currentMatchedRequestNode);
     if (!newMatchedRequestIndex && jumpBackwards)
       newMatchedRequestIndex = this._matchedRequestCount - 1;
     this._highlightNthMatchedRequestForSearch(newMatchedRequestIndex, shouldJump);
   }
 
   /**
    * @override
    * @return {boolean}
    */
@@ skipping 38 matching lines @@
    */
   _normalizeSearchResultIndex(index) {
     return (index + this._matchedRequestCount) % this._matchedRequestCount;
   }
 
   /**
    * @param {!Network.NetworkDataGridNode} node
    * @return {boolean}
    */
   _applyFilter(node) {
-    var request = node.request();
-    if (this._timeFilter && !this._timeFilter(request))
-      return false;
-    var categoryName = request.resourceType().category().title;
-    if (!this._resourceCategoryFilterUI.accept(categoryName))
-      return false;
-    if (this._dataURLFilterUI.checked() && request.parsedURL.isDataURL())
-      return false;
-    if (request.statusText === 'Service Worker Fallback Required')
-      return false;
-    for (var i = 0; i < this._filters.length; ++i) {
-      if (!this._filters[i](request))
+    for (var request of node.requests()) {

[dgozman, 2016/11/23 01:57:30]

This is incorrect: if one request matches, we should leave the whole node.

[allada, 2016/11/23 22:39:14]

Done.

+      if (this._timeFilter && !this._timeFilter(request))
         return false;
+      var categoryName = request.resourceType().category().title;
+      if (!this._resourceCategoryFilterUI.accept(categoryName))
+        return false;
+      if (this._dataURLFilterUI.checked() && request.parsedURL.isDataURL())
+        return false;
+      if (request.statusText === 'Service Worker Fallback Required')
+        return false;
+      for (var i = 0; i < this._filters.length; ++i) {
+        if (!this._filters[i](request))
+          return false;
+      }
     }
     return true;
   }
 
   /**
    * @param {string} query
    */
   _parseFilterQuery(query) {
     var parsedQuery;
     if (this._textFilterUI.isRegexChecked() && query !== '')
@@ skipping 349 matching lines @@
   Running: 'running',
   FromCache: 'from-cache'
 };
 
 /** @type {!Array<string>} */
 Network.NetworkLogView._searchKeys =
     Object.keys(Network.NetworkLogView.FilterType).map(key => Network.NetworkLogView.FilterType[key]);
 
 /** @typedef {function(!SDK.NetworkRequest): boolean} */
 Network.NetworkLogView.Filter;