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

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

Issue 2839273003: [Devtools] New structure and colorize rows for network products (Closed)
Patch Set: Merge remote-tracking branch 'origin/master' into NEW_DEPENDENCY_PRODUCTS Created 3 years, 8 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 7e8d4a06d4e88ea0d8a449ab558c212034fcc81e..b0e51d3acfa926c65bb04c92f552f4f6b59605ac 100644
--- a/third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js
+++ b/third_party/WebKit/Source/devtools/front_end/network/NetworkLogView.js
@@ -57,9 +57,6 @@ Network.NetworkLogView = class extends UI.VBox {
this._durationCalculator = new Network.NetworkTransferDurationCalculator();
this._calculator = this._timeCalculator;
- /** @type {?Network.NetworkGroupLookupInterface} */
- this._activeGroupLookup = null;
-
/**
* @this {Network.NetworkLogView}
*/
@@ -76,9 +73,6 @@ Network.NetworkLogView = class extends UI.VBox {
this._nodesByRequestId = new Map();
/** @type {!Map<*, !Network.NetworkGroupNode>} */
this._nodeGroups = new Map();
- /** @type {!Set<!Network.NetworkRowDecorator>} */
- this._rowDecorators = new Set();
-
/** @type {!Object.<string, boolean>} */
this._staleRequestIds = {};
/** @type {number} */
@@ -106,6 +100,14 @@ Network.NetworkLogView = class extends UI.VBox {
this._headerHeight = 0;
+ /** @type {!Map<string, !Network.GroupLookupInterface>} */
+ this._groupLookups = new Map();
+ this._groupLookups.set('Product', new Network.ProductGrouper());
+ this._groupLookups.set('Frame', new Network.FrameGrouper());
+
+ /** @type {?Network.GroupLookupInterface} */
+ this._activeGroupLookup = null;
+
this._addFilters();
this._resetSuggestionBuilder();
this._initializeView();
@@ -352,19 +354,29 @@ Network.NetworkLogView = class extends UI.VBox {
}
/**
- * @param {?Network.NetworkGroupLookupInterface} grouping
+ * @return {!Map<string, !Network.GroupLookupInterface>}
*/
- setGrouping(grouping) {
- this._activeGroupLookup = grouping;
- this._nodeGroups.clear();
- this._invalidateAllItems();
+ groupLookups() {
+ return this._groupLookups;
}
/**
- * @return {!Set<!Network.NetworkRowDecorator>}
+ * @param {string} groupKey
*/
- rowDecorators() {
- return this._rowDecorators;
+ setGrouping(groupKey) {
+ var groupLookup = this._groupLookups.get(groupKey) || null;
+ this._activeGroupLookup = groupLookup;
+ if (!groupLookup) {
+ this._nodeGroups.clear();
+ this._invalidateAllItems();
+ return;
+ }
+ groupLookup.initialize().then(() => {
+ if (this._activeGroupLookup !== groupLookup)
+ return;
+ this._nodeGroups.clear();
+ this._invalidateAllItems();
+ });
}
/**
@@ -498,12 +510,6 @@ Network.NetworkLogView = class extends UI.VBox {
this.element.id = 'network-container';
this._setupDataGrid();
- self.runtime.allInstances(Network.NetworkRowDecorator).then(instances => {
- for (var instance of instances)
- this._rowDecorators.add(instance);
- this._invalidateAllItems(true);
- });
-
this._columns.show(this.element);
this._summaryBarElement = this.element.createChild('div', 'network-summary-bar');
@@ -900,7 +906,6 @@ Network.NetworkLogView = class extends UI.VBox {
if (group)
return group;
group = new Network.NetworkGroupNode(this, this._activeGroupLookup.groupName(groupKey));
- group.setColumnExtensions(this._columns.columnExtensions());
this._nodeGroups.set(groupKey, group);
return group;
}
@@ -962,7 +967,6 @@ Network.NetworkLogView = class extends UI.VBox {
*/
_appendRequest(request) {
var node = new Network.NetworkRequestNode(this, request);
- node.setColumnExtensions(this._columns.columnExtensions());
node[Network.NetworkLogView._isFilteredOutSymbol] = true;
node[Network.NetworkLogView._isMatchingSearchQuerySymbol] = false;
@@ -1819,30 +1823,23 @@ Network.NetworkLogView.Filter;
/**
* @interface
*/
-Network.NetworkGroupLookupInterface = function() {};
+Network.GroupLookupInterface = function() {};
+
+Network.GroupLookupInterface.prototype = {
+ /**
+ * @return {!Promise}
+ */
+ initialize: function() {},
-Network.NetworkGroupLookupInterface.prototype = {
/**
* @param {!SDK.NetworkRequest} request
* @return {?*}
*/
- groupForRequest(request) {},
+ groupForRequest: function(request) {},
/**
* @param {!*} key
* @return {string}
*/
- groupName(key) {}
-};
-
-/**
- * @interface
- */
-Network.NetworkRowDecorator = function() {};
-
-Network.NetworkRowDecorator.prototype = {
- /**
- * @param {!Network.NetworkNode} node
- */
- decorate(node) {}
+ groupName: function(key) {}
};

Powered by Google App Engine
This is Rietveld 408576698