Title: [294588] trunk/Source/WebInspectorUI/UserInterface/Views/LocalResourceOverridePopover.js
- Revision
- 294588
- Author
- drou...@apple.com
- Date
- 2022-05-20 15:53:33 -0700 (Fri, 20 May 2022)
Log Message
Web Inspector: Sources Tab: adding a new header to a request local override doesn't show it
https://bugs.webkit.org/show_bug.cgi?id=240733
Reviewed by Patrick Angle.
* Source/WebInspectorUI/UserInterface/Views/LocalResourceOverridePopover.js:
(WI.LocalResourceOverridePopover.prototype.show):
Canonical link: https://commits.webkit.org/250821@main
Modified Paths
Diff
Modified: trunk/Source/WebInspectorUI/UserInterface/Views/LocalResourceOverridePopover.js (294587 => 294588)
--- trunk/Source/WebInspectorUI/UserInterface/Views/LocalResourceOverridePopover.js 2022-05-20 22:37:00 UTC (rev 294587)
+++ trunk/Source/WebInspectorUI/UserInterface/Views/LocalResourceOverridePopover.js 2022-05-20 22:53:33 UTC (rev 294588)
@@ -362,8 +362,10 @@
};
let toggleHeadersDataGridVisibility = (force) => {
- this._headersDataGrid.element.hidden = force !== undefined ? force : !this._headersDataGrid.hasChildren;
- this._headersDataGrid.updateLayoutIfNeeded();
+ let hidden = force ?? !this._headersDataGrid.hasChildren;
+ this._headersDataGrid.element.hidden = hidden;
+ if (!hidden)
+ this._headersDataGrid.updateLayout();
};
let contentTypeDataGridNode = addDataGridNodeForHeader(WI.unlocalizedString("Content-Type"), valueData.mimeType || placeholderData.mimeType, {selectable: false, editable: false, classNames: ["header-content-type"]});
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes