Fix to follow.
Created attachment 186608 [details] Patch
Comment on attachment 186608 [details] Patch Attachment 186608 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/16379608
Comment on attachment 186608 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=186608&action=review > Source/WebCore/inspector/front-end/FilteredItemSelectionDialog.js:288 > + this._updateSelection(this._selectedIndexInFiltered, false); Should we check this._selectedIndexInFiltered for null? > Source/WebCore/inspector/front-end/FilteredItemSelectionDialog.js:301 > + event.consume(); Wouldn't it be better to do event.consume(true) instead? > Source/WebCore/inspector/front-end/FilteredItemSelectionDialog.js:302 > + return; These "return; break;" pairs look really weird. I would move _scheduleFilter call to default section of this switch instead.
Created attachment 186788 [details] Patch
Committed r141979: <http://trac.webkit.org/changeset/141979>