Web Inspector: Annotators should not be enabled if the TextEditor is not visible
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 1 Mar 2017 23:59:55 +0000 (23:59 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 1 Mar 2017 23:59:55 +0000 (23:59 +0000)
commit34f37caa5b274c4e676ca10e00f6471f98685943
treec1054703e47a6fc48b2004c4b3d831a585dc413d
parent4e5cf9a746e91b12668f56217d9dfb696c7670d1
Web Inspector: Annotators should not be enabled if the TextEditor is not visible
https://bugs.webkit.org/show_bug.cgi?id=169059

Patch by Devin Rousso <dcrousso+webkit@gmail.com> on 2017-03-01
Reviewed by Joseph Pecoraro.

* UserInterface/Views/DebuggerSidebarPanel.js:
(WebInspector.DebuggerSidebarPanel.prototype._addResource):
(WebInspector.DebuggerSidebarPanel.prototype._addScript):
Only call `showDefaultContentViewForTreeElement` if the panel has been added to a sidebar.

It is not valid to simply check whether the panel is visible, as the sidebar can be collapsed.
In that case, we still want to display the content view, so we instead check that the panel
has been added to a sidebar, meaning that it is ready to be displayed in the current tab.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@213251 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Source/WebInspectorUI/ChangeLog
Source/WebInspectorUI/UserInterface/Views/DebuggerSidebarPanel.js