|Summary:||Web Inspector: The right sidebar always opens up on breakpoint|
|Product:||WebKit||Reporter:||Nikita Vasilyev <nvasilyev>|
|Component:||Web Inspector||Assignee:||Devin Rousso <hi>|
|Severity:||Normal||CC:||bburg, commit-queue, graouts, joepeck, mattbaker, nvasilyev, timothy, webkit-bug-importer|
|Version:||WebKit Local Build|
|Bug Depends on:||149284|
Description Nikita Vasilyev 2015-09-17 07:09:16 PDT
Created attachment 261391 [details] [Image] Current behavior The right sidebar always opens up when paused on a breakpoint, taking a significant amount of my screen space. Even if I close it, it will open again on a next breakpoint. When execution resumed, the sidebar still stays on, even if it's empty. At very least, we should close the sidebar when resuming execution and there are no watch expressions. I don't think it needs to be open every time since it doesn't even provide essential debugger buttons, e.g. resume, step in, etc. Maybe we should remember if it was closed while paused on debugger and not open it again next time.
Comment 2 Timothy Hatcher 2015-09-17 08:42:51 PDT
Matt was wanting to change this too.
Comment 3 Devin Rousso 2015-09-17 13:18:34 PDT
Created attachment 261413 [details] Patch This fixes the issue for me. One quick question though: since I removed the only instance that showScopeChainDetailsSidebarPanel is ever called, should I also remove that function and its corresponding logic or leave it there in case it is needed again?
Comment 4 Joseph Pecoraro 2015-09-17 13:48:54 PDT
Comment on attachment 261413 [details] Patch cq- for now. 1. We think it would be good to add a Preference for this. We should add a FIXME: <webkit.org/b/###> Web Inspector: Preference for Auto Showing Scope Chain sidebar on pause 2. This should remove the "showScopeChainDetailsSidebarPanel" parameter, it is no longer being used. 823:WebInspector.showDebuggerTab = function(breakpointToSelect, showScopeChainDetailsSidebarPanel)
Comment 6 WebKit Commit Bot 2015-09-22 18:29:55 PDT
Comment on attachment 261785 [details] Patch Clearing flags on attachment: 261785 Committed r190149: <http://trac.webkit.org/changeset/190149>
Comment 7 WebKit Commit Bot 2015-09-22 18:30:00 PDT
All reviewed patches have been landed. Closing bug.