[DevTools] Fix more frontend compiler errors

third_party/WebKit/Source/devtools/front_end/timeline/TimelineTreeView.js

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * @constructor
  * @extends {WebInspector.VBox}
  * @param {!WebInspector.TimelineModel} model
  */
 WebInspector.TimelineTreeView = function(model)
@@ skipping 139 matching lines @@
     _buildTopDownTree: function(eventIdCallback)
     {
         return WebInspector.TimelineProfileTree.buildTopDown(this._model.mainThreadEvents(), this._filters, this._startTime, this._endTime, eventIdCallback)
     },
 
     /**
      * @param {!Array.<!WebInspector.DataGrid.ColumnDescriptor>} columns
      */
     _populateColumns: function(columns)
     {
-        columns.push({id: "self", title: WebInspector.UIString("Self Time"), width: "110px", fixedWidth: true, sortable: true});
-        columns.push({id: "total", title: WebInspector.UIString("Total Time"), width: "110px", fixedWidth: true, sortable: true});
-        columns.push({id: "activity", title: WebInspector.UIString("Activity"), disclosure: true, sortable: true});
+        columns.push({id: "self", title: WebInspector.UIString("Self Time"), width: "110px", fixedWidth: true, sortable: true, editable: false, longText: null, nonSelectable: false, align: null, sort: null});

[lushnikov, 2016/03/04 22:49:18]

lets use constructor with optional parameters?

[kozy, 2016/03/04 23:18:50]

introduced struct instead

+        columns.push({id: "total", title: WebInspector.UIString("Total Time"), width: "110px", fixedWidth: true, sortable: true, editable: false, longText: null, nonSelectable: false, align: null, sort: null});
+        columns.push({id: "activity", title: WebInspector.UIString("Activity"), disclosure: true, sortable: true, editable: false, longText: null, nonSelectable: false, align: null, sort: null});
     },
 
     _sortingChanged: function()
     {
         var columnIdentifier = this._dataGrid.sortColumnIdentifier();
         if (!columnIdentifier)
             return;
         var sortFunction;
         switch (columnIdentifier) {
         case "startTime":
@@ skipping 718 matching lines @@
         if (expand)
             this._dataGridNodeForTreeNode(node).expand();
     },
 
     /**
      * @override
      * @param {!Array<!WebInspector.DataGrid.ColumnDescriptor>} columns
      */
     _populateColumns: function(columns)
     {
-        columns.push({id: "startTime", title: WebInspector.UIString("Start Time"), width: "110px", fixedWidth: true, sortable: true});
+        columns.push({id: "startTime", title: WebInspector.UIString("Start Time"), width: "110px", fixedWidth: true, sortable: true, align: null, editable: false, longText: null, nonSelectable: false, sort: null});
         WebInspector.TimelineTreeView.prototype._populateColumns.call(this, columns);
     },
 
     /**
      * @override
      * @param {!Element} parent
      */
     _populateToolbar: function(parent)
     {
         var filtersWidget = this._filtersControl.filtersWidget();
@@ skipping 92 matching lines @@
         return selectedNode && /** @type {!WebInspector.TimelineTreeView.GridNode} */ (selectedNode)._profileNode;
     },
 
     _onSelectionChanged: function()
     {
         this.dispatchEventToListeners(WebInspector.TimelineStackView.Events.SelectionChanged);
     },
 
     __proto__: WebInspector.VBox.prototype
 }