DevTools: WebSocketFrameView - Add frames display control toolbar

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

 /*
  * Copyright (C) 2012 Research In Motion Limited. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Lesser General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 26 matching lines @@
         id: 'length',
         title: Common.UIString('Length'),
         sortable: false,
         align: DataGrid.DataGrid.Align.Right,
         weight: 5
       },
       {id: 'time', title: Common.UIString('Time'), sortable: true, weight: 7}
     ]);
 
     this._dataGrid = new DataGrid.SortableDataGrid(columns);
-    this._dataGrid.setRowContextMenuCallback(onRowContextMenu);
+    this._dataGrid.setRowContextMenuCallback(onRowContextMenu.bind(this));
     this._dataGrid.setStickToBottom(true);
     this._dataGrid.setCellClass('websocket-frame-view-td');
     this._timeComparator =
         /** @type {function(!Network.ResourceWebSocketFrameNode, !Network.ResourceWebSocketFrameNode):number} */ (
             Network.ResourceWebSocketFrameNodeTimeComparator);
     this._dataGrid.sortNodes(this._timeComparator, false);
     this._dataGrid.markColumnAsSortedBy('time', DataGrid.DataGrid.Order.Ascending);
     this._dataGrid.addEventListener(DataGrid.DataGrid.Events.SortingChanged, this._sortItems, this);
 
     this._dataGrid.setName('ResourceWebSocketFrameView');
     this._dataGrid.addEventListener(DataGrid.DataGrid.Events.SelectedNode, this._onFrameSelected, this);
     this._dataGrid.addEventListener(DataGrid.DataGrid.Events.DeselectedNode, this._onFrameDeselected, this);
-    this._splitWidget.setMainWidget(this._dataGrid.asWidget());
 
-    var view = new UI.EmptyWidget('Select frame to browse its content.');
-    this._splitWidget.setSidebarWidget(view);
+    this._clearAllButton = new UI.ToolbarButton(Common.UIString('Clear All'), 'largeicon-clear');
+    this._clearAllButton.addEventListener(UI.ToolbarButton.Events.Click, this._onClearAllWidgetClicked, this);
+    this._clearFrameOffset = 0;
+
+    this._textFilterUI = new UI.TextFilterUI(true);
+    this._textFilterUI.addEventListener(UI.FilterUI.Events.FilterChanged, this._onFilterUIChanged, this);

[allada, 2017/04/17 21:11:19]

Lets inline "_onFilterUIChanged"

ie: this.refresh

+
+    var typeFilterItems = [

[allada, 2017/04/17 21:11:19]

Lets move this to near the bottom of the file.

Something like:

Network.ResourceWebSocketFrameView._filterTypes = [...

+      {name: 'send', label: Common.UIString('Send')},
+      {name: 'receive', label: Common.UIString('Receive')},
+    ];
+    this._typeFilterSetting = Common.settings.createSetting('resourceWebSocketFrameViewTypeFilters', {});

[allada, 2017/04/17 21:11:19]

lets make this a local variable since it's not needed outside the constructor.

+    this._typeFilterUI = new UI.NamedBitSetFilterUI(typeFilterItems, this._typeFilterSetting);
+    this._typeFilterUI.addEventListener(UI.FilterUI.Events.FilterChanged, this._onFilterUIChanged, this);

[allada, 2017/04/17 21:11:19]

Same here... inline "_onFilterUIChanged"

ie: this.refresh

+
+    this._filterBar = new UI.FilterBar('resourceWebSocketFrameView', true);

[allada, 2017/04/17 21:11:19]

lets rename this just "websocketFrameView". (resources and network should now be
fully separated)

+    this._filterBar.addFilter(this._textFilterUI);

[allada, 2017/04/17 21:11:19]

Did you mean to add this twice?

[muzuiget, 2017/04/20 09:40:01]

Sorry, I didn't get your point. One is "_textFilterUI" another is
"_typeFilterUI"

[allada, 2017/04/24 18:28:14]

I see.

+    this._filterBar.addFilter(this._typeFilterUI);
+
+    this._mainToolbar = new UI.Toolbar('');
+    this._mainToolbar.appendToolbarItem(this._clearAllButton);
+    this._mainToolbar.appendToolbarItem(this._filterBar.filterButton());
+
+    var mainContainer = new UI.VBox();
+    mainContainer.element.appendChild(this._mainToolbar.element);
+    this._filterBar.show(mainContainer.element);
+    this._dataGrid.asWidget().show(mainContainer.element);
+    this._splitWidget.setMainWidget(mainContainer);
+
+    this._frameEmptyWidget = new UI.EmptyWidget('Select frame to browse its content.');

[allada, 2017/04/17 21:11:19]

I know this was not how it was originally, but can you change this line to:
this._frameEmptyWidget = new UI.EmptyWidget(Common.UIString('Select frame to
browse its content.'));

+    this._splitWidget.setSidebarWidget(this._frameEmptyWidget);
 
     /** @type {?Network.ResourceWebSocketFrameNode} */
     this._selectedNode = null;
 
     /**
      * @param {!UI.ContextMenu} contextMenu
      * @param {!DataGrid.DataGridNode} node
+     * @this {Network.ResourceWebSocketFrameView}
      */
     function onRowContextMenu(contextMenu, node) {
       contextMenu.appendItem(
           Common.UIString.capitalize('Copy ^message'),
           InspectorFrontendHost.copyText.bind(InspectorFrontendHost, node.data.data));
+      contextMenu.appendSeparator();
+      contextMenu.appendItem(Common.UIString.capitalize('Clear ^all'), this._onClearAllWidgetClicked.bind(this));
     }
   }
 
   /**
    * @param {number} opCode
    * @param {boolean} mask
    * @return {string}
    */
   static opCodeDescription(opCode, mask) {
     var rawDescription = Network.ResourceWebSocketFrameView.opCodeDescriptions[opCode] || '';
@@ skipping 14 matching lines @@
    */
   willHide() {
     this._request.removeEventListener(SDK.NetworkRequest.Events.WebsocketFrameAdded, this._frameAdded, this);
   }
 
   /**
    * @param {!Common.Event} event
    */
   _frameAdded(event) {
     var frame = /** @type {!SDK.NetworkRequest.WebSocketFrame} */ (event.data);
+    if (!this._frameFilter(frame))
+      return;
     this._dataGrid.insertChild(new Network.ResourceWebSocketFrameNode(this._request.url(), frame));
   }
 
   /**
+   * @param {!SDK.NetworkRequest.WebSocketFrame} frame
+   */
+  _frameFilter(frame) {
+    var regex = this._textFilterUI.regex();

[allada, 2017/04/17 21:11:19]

In this case regex should be the tester used for both when regex is checked and
not.

In other words this should be all you need:

return regex && regex.test(frame.text) && this._typeFilterUI.accept(frame.type);

+    if (regex) {
+      if (!regex.test(frame.text))
+        return false;
+    } else {
+      var text = this._textFilterUI.value();
+      if (!frame.text.includes(text))
+        return false;
+    }
+
+    if (!this._typeFilterUI.accept(frame.type))
+      return false;
+
+    return true;
+  }
+
+  /**
+   * @param {!Common.Event} event
+   */
+  _onClearAllWidgetClicked(event) {

[allada, 2017/04/17 21:11:19]

Lets rename this "_clearFrames" and remove the event argument, so it's easier to
reuse.

+    this._clearFrameOffset = this._request.frames().length;
+    this.refresh();
+  }
+
+  /**
+   * @param {!Common.Event} event
+   */
+  _onFilterUIChanged(event) {
+    this.refresh();
+  }
+
+  /**
    * @param {!Common.Event} event
    */
   _onFrameSelected(event) {
     var selectedNode = /** @type {!Network.ResourceWebSocketFrameNode} */ (event.data);
     this._currentSelectedNode = selectedNode;
     var contentProvider = selectedNode.contentProvider();
     contentProvider.requestContent().then(contentHandler.bind(this));
 
     /**
      * @param {(string|null)} content
@@ skipping 17 matching lines @@
       else
         this._splitWidget.setSidebarWidget(new SourceFrame.ResourceSourceFrame(contentProvider));
     }
   }
 
   /**
    * @param {!Common.Event} event
    */
   _onFrameDeselected(event) {
     this._currentSelectedNode = null;
+    this._splitWidget.setSidebarWidget(this._frameEmptyWidget);
   }
 
   refresh() {
     this._dataGrid.rootNode().removeChildren();
+
+    var url = this._request.url();
     var frames = this._request.frames();
-    for (var i = 0; i < frames.length; ++i)
-      this._dataGrid.insertChild(new Network.ResourceWebSocketFrameNode(this._request.url(), frames[i]));
+    var start = this._clearFrameOffset;
+    for (var i = start; i < frames.length; ++i) {

[allada, 2017/04/17 21:11:19]

Can we change this to:

frames =
frames.slice(this._clearFrameOffset).filter(this._frameFilter.bind(this));
frames.forEach(frame => this._dataGrid.insertChild(new
Network.ResourceWebSocketFrameNode(url, frame)));

+      var frame = frames[i];
+      if (this._frameFilter(frame))
+        this._dataGrid.insertChild(new Network.ResourceWebSocketFrameNode(url, frame));
+    }
   }
 
   _sortItems() {
     this._dataGrid.sortNodes(this._timeComparator, !this._dataGrid.isSortOrderAscending());
   }
 };
 
 /** @enum {number} */
 Network.ResourceWebSocketFrameView.OpCodes = {
   ContinuationFrame: 0,
@@ skipping 78 matching lines @@
 };
 
 /**
  * @param {!Network.ResourceWebSocketFrameNode} a
  * @param {!Network.ResourceWebSocketFrameNode} b
  * @return {number}
  */
 Network.ResourceWebSocketFrameNodeTimeComparator = function(a, b) {
   return a._frame.time - b._frame.time;
 };