Moving the width-saving WI.Setting logic to the WI.TabBrowser caused the navigation sidebar to no longer remember its width.
Created attachment 296341 [details] Patch
Comment on attachment 296341 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=296341&action=review Looks good! r=me with nits > Source/WebInspectorUI/UserInterface/Views/TabBrowser.js:299 > + _sidebarWidthDidChange(event) { Open brace should be on its own line. > Source/WebInspectorUI/UserInterface/Views/TabContentView.js:51 > + this._navigationSidebarWidthSetting = new WebInspector.Setting(identifier + "-navigation-sidebar-width", 300); Since 300 is used in two places lets make it a const: const defaultSidebarWidth = 300;
Created attachment 296448 [details] Patch
Comment on attachment 296448 [details] Patch Clearing flags on attachment: 296448 Committed r209519: <http://trac.webkit.org/changeset/209519>
All reviewed patches have been landed. Closing bug.