Timeline: add moar attributes in frame instrumentation

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.
 /**
  * @unrestricted
  */
 WebInspector.TimelineTreeView = class extends WebInspector.VBox {
   constructor() {
     super();
     this.element.classList.add('timeline-tree-view');
@@ skipping 138 matching lines @@
   }
 
   /**
    * @return {!WebInspector.TimelineProfileTree.Node}
    */
   _buildTree() {
     throw new Error('Not Implemented');
   }
 
   /**
-   * @param {function(!WebInspector.TracingModel.Event, string):(string|symbol)=} eventIdCallback
+   * @param {function(!WebInspector.TracingModel.Event):(string|symbol)=} eventIdCallback
    * @return {!WebInspector.TimelineProfileTree.Node}
    */
   _buildTopDownTree(eventIdCallback) {
     return WebInspector.TimelineProfileTree.buildTopDown(
         this._model.mainThreadEvents(), this._filters, this._startTime, this._endTime, eventIdCallback);
   }
 
   /**
    * @param {!Array<!WebInspector.DataGrid.ColumnDescriptor>} columns
    */
@@ skipping 434 matching lines @@
    */
   _showDetailsForNode(node) {
     var stack = this._buildHeaviestStack(node);
     this._stackView.setStack(stack, node);
     this._stackView.show(this._detailsView.element);
     return true;
   }
 
   /**
    * @param {!WebInspector.AggregatedTimelineTreeView.GroupBy} groupBy
-   * @return {function(!WebInspector.TracingModel.Event, string):string}
+   * @return {function(!WebInspector.TracingModel.Event):string}
    */
   _groupingFunction(groupBy) {
     /**
      * @param {!WebInspector.TracingModel.Event} event
      * @return {string}
      */
     function groupByURL(event) {
       return WebInspector.TimelineProfileTree.eventURL(event) || '';
     }
 
@@ skipping 26 matching lines @@
         return event => WebInspector.TimelineUIUtils.eventStyle(event).title;
       case WebInspector.AggregatedTimelineTreeView.GroupBy.Category:
         return event => WebInspector.TimelineUIUtils.eventStyle(event).category.name;
       case WebInspector.AggregatedTimelineTreeView.GroupBy.Subdomain:
         return groupByDomain.bind(null, false);
       case WebInspector.AggregatedTimelineTreeView.GroupBy.Domain:
         return groupByDomain.bind(null, true);
       case WebInspector.AggregatedTimelineTreeView.GroupBy.URL:
         return groupByURL;
       case WebInspector.AggregatedTimelineTreeView.GroupBy.Frame:
-        return (event, pageFrameId) => pageFrameId || '';
+        return event => WebInspector.TimelineData.forEvent(event).frameId;
       default:
         console.assert(false, `Unexpected aggregation setting: ${groupBy}`);
         return () => Symbol('uniqueGroupId');
     }
   }
 
   /**
    * @param {string} url
    * @return {boolean}
    */
@@ skipping 252 matching lines @@
 
   _onSelectionChanged() {
     this.dispatchEventToListeners(WebInspector.TimelineStackView.Events.SelectionChanged);
   }
 };
 
 /** @enum {symbol} */
 WebInspector.TimelineStackView.Events = {
   SelectionChanged: Symbol('SelectionChanged')
 };