Use HashMap::remove() in some places that I wanted it.

Author: https://github.com/awesomekling
Commit: https://github.com/SerenityOS/serenity/commit/6ea8ce500c5

VirtualFileSystem/FileSystem.cpp

@@ -18,8 +18,7 @@ FileSystem::FileSystem()
 
 FileSystem::~FileSystem()
 {
-    // FIXME: Needs HashMap::remove()!
-    //fileSystems().remove(m_id);
+    fileSystems().remove(m_id);
 }
 
 FileSystem* FileSystem::fromID(dword id)

VirtualFileSystem/VirtualFileSystem.cpp

@@ -115,11 +115,10 @@ void VirtualFileSystem::freeNode(Node* node)
 {
     ASSERT(node);
     ASSERT(node->inUse());
+    m_inode2vnode.remove(node->inode);
     node->inode.fileSystem()->release();
     node->inode = InodeIdentifier();
     m_nodeFreeList.append(std::move(node));
-    // FIXME: Need HashMap::remove.
-    //m_inode2vnode.remove(node);
 }
 
 bool VirtualFileSystem::isDirectory(const String& path)

Widgets/Window.cpp

@@ -11,6 +11,7 @@ Window::Window(Object* parent)
 
 Window::~Window()
 {
+    WindowManager::the().removeWindow(*this);
 }
 
 void Window::setMainWidget(Widget* widget)
@@ -31,7 +32,6 @@ void Window::setTitle(String&& title)
     WindowManager::the().notifyTitleChanged(*this);
 }
 
-
 void Window::setRect(const Rect& rect)
 {
     if (m_rect == rect)

Widgets/WindowManager.cpp

@@ -118,6 +118,12 @@ void WindowManager::addWindow(Window& window)
     m_windows.set(&window);
 }
 
+void WindowManager::removeWindow(Window& window)
+{
+    ASSERT(m_windows.contains(&window));
+    m_windows.remove(&window);
+}
+
 void WindowManager::notifyTitleChanged(Window& window)
 {
     //printf("[WM] Window{%p} title set to '%s'\n", &window, window.title().characters());

Widgets/WindowManager.h

@@ -14,6 +14,7 @@ class WindowManager : public Object {
 public:
     static WindowManager& the(); 
     void addWindow(Window&);
+    void removeWindow(Window&);
     void paintWindowFrames();
 
     void notifyTitleChanged(Window&);