(Based on an IRC conversation with JoePeck.) Although some views have a single persistent instance, ideally any event listeners attached to managers and the like should still be cleaned up in closed().
Created attachment 319015 [details] Patch
Comment on attachment 319015 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=319015&action=review r=me. Please rebase and upload a new patch to EWS before landing. > Source/WebInspectorUI/UserInterface/Views/QuickConsole.js:106 > + Technically outside the scope of this patch, but this should also unregister from WI.consoleDrawer and WI.TabBrowser.
Created attachment 323327 [details] Patch
Comment on attachment 323327 [details] Patch Clearing flags on attachment: 323327 Committed r223137: <http://trac.webkit.org/changeset/223137>
All reviewed patches have been landed. Closing bug.
<rdar://problem/34916512>