withSocket listeners not being properly removed

[lolney]

To reproduce:

• Open the siege items browser, then close it
• Trigger a resource update event, e.g. when a bot comes back to base

Triggers the warning message: Warning: Can't call setState (or forceUpdate) on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in the componentWillUnmount method. in WithSocket (created by WithSocketFetch) in WithSocketFetch in div (created by Window) in div (created by Window) in DraggableCore (created by Draggable) in Draggable (created by Window) in Window

Listeners are being removed here, though.