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

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

Issue 2839273003: [Devtools] New structure and colorize rows for network products (Closed)
Patch Set: changes 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/NetworkLogViewColumns.js
diff --git a/third_party/WebKit/Source/devtools/front_end/network/NetworkLogViewColumns.js b/third_party/WebKit/Source/devtools/front_end/network/NetworkLogViewColumns.js
index e206ff068489418f3909c338c3ad97cb5f7a4ce4..96d19c5a06aa5093b92feb47d449831d27d95f51 100644
--- a/third_party/WebKit/Source/devtools/front_end/network/NetworkLogViewColumns.js
+++ b/third_party/WebKit/Source/devtools/front_end/network/NetworkLogViewColumns.js
@@ -29,12 +29,12 @@ Network.NetworkLogViewColumns = class {
/** @type {!Array.<!Network.NetworkLogViewColumns.Descriptor>} */
this._columns = [];
- /** @type {!Map<string, ?Network.NetworkColumnExtensionInterface>} */
- this._columnExtensions = new Map();
-
this._waterfallRequestsAreStale = false;
this._waterfallScrollerWidthIsStale = true;
+ /** @type {?ProductRegistry.Registry} */
+ this._productRegistryInstance = null;
+
/** @type {!Components.Linkifier} */
this._popupLinkifier = new Components.Linkifier();
@@ -78,6 +78,15 @@ Network.NetworkLogViewColumns = class {
_setupDataGrid() {
var defaultColumns = Network.NetworkLogViewColumns._defaultColumns;
+
+ if (Runtime.experiments.isEnabled('networkGroupingRequests')) {
+ defaultColumns.splice(1, 0, /** @type {!Network.NetworkLogViewColumns.Descriptor} */ ({
+ id: 'product',
+ title: Common.UIString('Product'),
+ visible: true
+ }));
+ }
+
var defaultColumnConfig = Network.NetworkLogViewColumns._defaultColumnConfig;
this._columns = /** @type {!Array<!Network.NetworkLogViewColumns.Descriptor>} */ ([]);
@@ -89,7 +98,6 @@ Network.NetworkLogViewColumns = class {
columnConfig.titleDOMFragment = this._makeHeaderFragment(columnConfig.title, columnConfig.subtitle);
this._columns.push(columnConfig);
}
- this._loadColumnExtensions();
this._loadCustomColumnsAndSettings();
this._popoverHelper = new UI.PopoverHelper(this._networkLogView.element, this._getPopoverRequest.bind(this));
@@ -279,6 +287,17 @@ Network.NetworkLogViewColumns = class {
this._dataGrid.sortNodes(sortFunction, !this._dataGrid.isSortOrderAscending());
this._networkLogView.dataGridSorted();
return;
+ } else if (columnId === 'product' && !this._productRegistryInstance) {
+ ProductRegistry.instance().then(productRegistry => {
+ this._productRegistryInstance = productRegistry;
+ var columnConfig = this._columns.find(columnConfig => columnConfig.id === columnId);
+ if (!columnConfig)
+ return;
+ columnConfig.sortingFunction = Network.NetworkRequestNode.ProductComparator.bind(null, productRegistry);
+ if (this._dataGrid.sortColumnId() === 'product')
+ this._sortHandler();
+ });
+ return;
}
this._waterfallColumnSortIcon.setIconType('');
@@ -329,13 +348,6 @@ Network.NetworkLogViewColumns = class {
}
/**
- * @return {!Map<string, ?Network.NetworkColumnExtensionInterface>}
- */
- columnExtensions() {
- return this._columnExtensions;
- }
-
- /**
* @param {!Network.NetworkLogViewColumns.Descriptor} columnConfig
*/
_toggleColumnVisibility(columnConfig) {
@@ -353,43 +365,6 @@ Network.NetworkLogViewColumns = class {
this._persistantSettings.set(saveableSettings);
}
- _loadColumnExtensions() {
- var extensions = self.runtime.extensions(Network.NetworkColumnExtensionInterface);
- for (var i = 0; i < extensions.length; i++) {
- var extension = extensions[i];
- var title = extension.title();
- var columnId = title.toLowerCase() + '-extension';
-
- this._columnExtensions.set(columnId, null);
- extension.instance().then(extensionInstanceResolved.bind(this, columnId));
-
- var columnConfig = /** @type {!Network.NetworkLogViewColumns.Descriptor} */ (
- Object.assign({}, Network.NetworkLogViewColumns._defaultColumnConfig, {
- id: columnId,
- title: title,
- isResponseHeader: false,
- isCustomHeader: false,
- visible: true,
- sortingFunction:
- Network.NetworkRequestNode.ExtensionColumnComparator.bind(null, this._columnExtensions, columnId)
- }));
- const columnIndex = i + 1;
- this._columns.splice(columnIndex, 0, columnConfig);
- if (this._dataGrid)
- this._dataGrid.addColumn(Network.NetworkLogViewColumns._convertToDataGridDescriptor(columnConfig), columnIndex);
- }
-
- /**
- * @param {string} columnId
- * @param {!Network.NetworkColumnExtensionInterface} instance
- * @this {Network.NetworkLogViewColumns}
- */
- function extensionInstanceResolved(columnId, instance) {
- this._columnExtensions.set(columnId, instance);
- this._networkLogView.columnExtensionResolved();
- }
- }
-
_loadCustomColumnsAndSettings() {
var savedSettings = this._persistantSettings.get();
var columnIds = Object.keys(savedSettings);
@@ -642,6 +617,8 @@ Network.NetworkLogViewColumns = class {
};
Network.NetworkLogViewColumns._initialSortColumn = 'waterfall';
+/** @type {?ProductRegistry.Registry} */
+Network.NetworkRequestNode._productRegistryInstance = null;
/**
* @typedef {{
@@ -854,23 +831,3 @@ Network.NetworkLogViewColumns.WaterfallSortIds = {
Duration: 'duration',
Latency: 'latency'
};
-
-/**
- * @interface
- */
-Network.NetworkColumnExtensionInterface = function() {};
-
-Network.NetworkColumnExtensionInterface.prototype = {
- /**
- * @param {!SDK.NetworkRequest} request
- * @return {string}
- */
- lookupColumnValue(request) {},
-
- /**
- * @param {!SDK.NetworkRequest} aRequest
- * @param {!SDK.NetworkRequest} bRequest
- * @return {number}
- */
- requestComparator(aRequest, bRequest) {}
-};

Powered by Google App Engine
This is Rietveld 408576698