Bug 242186

Summary: Web Inspector: REGRESSION (Bug 241817 ) Filter bars missing from Styles and Computed panels
Product: WebKit Reporter: Razvan Caliman <rcaliman>
Component: Web InspectorAssignee: Razvan Caliman <rcaliman>
Status: RESOLVED FIXED    
Severity: Normal CC: inspector-bugzilla-changes, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   
See Also: https://bugs.webkit.org/show_bug.cgi?id=241817
Attachments:
Description Flags
Missing filter bars
none
With filter bars none

Description Razvan Caliman 2022-06-30 06:44:47 PDT 
Created attachment 460578 [details]
Missing filter bars

Regressed by Bug 241817

`Source/WebInspectorUI/UserInterface/Views/GeneralStyleDetailsSidebarPanel.js:216`

```
this.contentView.element.classList.toggle("has-filter-bar", !!this._filterBar);
```

The check for the truthiness of `this._filterBar` always occurs before the member is assigned. 
It always returns false: `!!undefined === false`.


`WI.GeneralStyleDetailsSidebarPanel._showPanel()` is called at the top of `WI.GeneralStyleDetailsSidebarPanel.initialLayout()` before `this._filterBar` is ever assigned.
Comment 1 Radar WebKit Bug Importer 2022-06-30 06:45:58 PDT 
<rdar://problem/96215062>
Comment 2 Razvan Caliman 2022-06-30 06:52:18 PDT 
Created attachment 460579 [details]
With filter bars
Comment 3 Razvan Caliman 2022-06-30 07:13:22 PDT 
Pull request: https://github.com/WebKit/WebKit/pull/1946
Comment 4 EWS 2022-07-02 04:17:11 PDT 
Committed 252093@main (27afcd755691): <https://commits.webkit.org/252093@main>

Reviewed commits have been landed. Closing PR #1946 and removing active labels.