(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]
Comment on attachment 319015 [details]
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.
Technically outside the scope of this patch, but this should also unregister from WI.consoleDrawer and WI.TabBrowser.
Created attachment 323327 [details]
Comment on attachment 323327 [details]
Clearing flags on attachment: 323327
Committed r223137: <http://trac.webkit.org/changeset/223137>
All reviewed patches have been landed. Closing bug.