Bug 135740 – Web Inspector: Timeline Filter Bars are not appearing at all

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector

[Joseph Pecoraro]

* SUMMARY
Timelines used to have filter bars, and they aren't showing at all. The code is still there, looks like an accident.

* STEPS TO REPRODUCE
1. Inspect apple.com
2. Reload
3. Go to Network Timeline
  => should be able to filter just images

[Radar WebKit Bug Importer]

<rdar://problem/17955461>

[Joseph Pecoraro]

Created attachment 236260 [details]
[PATCH] Proposed Fix

[Joseph Pecoraro]

Comment on attachment 236260 [details]
[PATCH] Proposed Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=236260&action=review

> Source/WebInspectorUI/UserInterface/Views/TimelineDataGrid.js:38
> -        var scopeBar = columns["scopeBar"];
> +        var scopeBar = column.scopeBar;

This is the actual fix. At some point "column" accidentally became "columns".

[Brian Burg]

Comment on attachment 236260 [details]
[PATCH] Proposed Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=236260&action=review

>> Source/WebInspectorUI/UserInterface/Views/TimelineDataGrid.js:38
>> +        var scopeBar = column.scopeBar;
> 
> This is the actual fix. At some point "column" accidentally became "columns".

Mea culpa, probably happened when column data went from being a map back to being an object. In lieu of UI tests, it would be nice to have a checklist of major functionality to check for each content view.

> Source/WebInspectorUI/UserInterface/Views/TimelineDataGrid.js:42
> +        scopeBar.columnIdentifier = identifier;

ouch.

[Brian Burg]

Comment on attachment 236260 [details]
[PATCH] Proposed Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=236260&action=review

> Source/WebInspectorUI/UserInterface/Views/ScopeBar.js:67
> +        for (var item of this._items) {

Or, go FP and do:

var defaultItem = this._defaultItem;
return this._items.some(function(item) { return item.selected && item !== defaultItem; })

> Source/WebInspectorUI/UserInterface/Views/TimelineDataGrid.js:364
> +                this.element.classList.toggle(WebInspector.TimelineDataGrid.HasNonDefaultFilterStyleClassName, scopeBar.hasNonDefaultItemSelected());

Mmm, toggle. I like toggle.

[Joseph Pecoraro]

Created attachment 236266 [details]
[PATCH] Proposed Fix

I agree with the Array.prototype.some(callback, [thisObject]) suggestion. I wanted to do that.

[WebKit Commit Bot]

Comment on attachment 236266 [details]
[PATCH] Proposed Fix

Clearing flags on attachment: 236266

Committed r172334: <http://trac.webkit.org/changeset/172334>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.