Remove deprecated methods

Change-Id: I1f04df7e9dd8dc0e7e2feac701d614a8a888b13c
Reviewed-on: http://gerrit.dmdirc.com/3013
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

src/com/dmdirc/FrameContainer.java

@@ -328,8 +328,6 @@ public abstract class FrameContainer {
         for (FrameCloseListener listener : listeners.get(FrameCloseListener.class)) {
             listener.windowClosing(this);
         }
-
-        windowClosing();
     }
 
     /**
@@ -397,41 +395,6 @@ public abstract class FrameContainer {
         }
     }
 
-    /**
-     * Invoked when our window is closing.
-     * <p>
-     * Frame containers must perform the following actions in this order:
-     * <ol>
-     *  <li>Remove any callbacks or listeners (events should not be processed
-     *      once a window has been requested to close)</li>
-     *  <li>Trigger any actions necessary (terminating any TCP connections,
-     *      disconnecting parsers, closing children, etc)</li>
-     *  <li>Trigger action for the window closing (raise a DMDirc action for
-     *      the closure of the window, if required)</li>
-     *  <li>Inform any parents that the window is closing (this includes
-     *      unregistering the window with any specific managers, or from the
-     *      parent windows if they track children)</li>
-     * </ol>
-     *
-     * @deprecated Clearing up should be performed in the {@link #close()} method.
-     */
-    @Deprecated
-    protected void windowClosing() {
-    }
-
-    /**
-     * Invoked when our window has been closed.
-     * <p>
-     * At this point, all interested parties have been told that the window
-     * has been closed, and therefore any references to frames or other
-     * resources may be completely freed.
-     *
-     * @deprecated Clearing up should be performed in the {@link #close()} method.
-     */
-    @Deprecated
-    public void windowClosed() {
-    }
-
     /**
      * Adds a line to this container's window. If the window is null for some
      * reason, the line is silently discarded.

src/com/dmdirc/ui/WindowManager.java

@@ -241,8 +241,6 @@ public class WindowManager {
             fireDeleteWindow(parent, window);
             parent.removeChild(window);
         }
-
-        window.windowClosed();
     }
 
     /**

test/com/dmdirc/ui/WindowManagerTest.java

@@ -142,7 +142,6 @@ public class WindowManagerTest {
         manager.removeWindow(container);
 
         verify(frameListener).delWindow(container);
-        verify(container).windowClosed();
     }
 
     @Test
@@ -157,7 +156,6 @@ public class WindowManagerTest {
 
         verify(frameListener).delWindow(container);
         verify(child).close();
-        verify(container).windowClosed();
     }
 
     @Test
@@ -171,7 +169,6 @@ public class WindowManagerTest {
         manager.removeWindow(child);
 
         verify(frameListener).delWindow(container, child);
-        verify(child).windowClosed();
         verify(container).removeChild(child);
     }
 
@@ -189,7 +186,6 @@ public class WindowManagerTest {
 
         verify(frameListener).delWindow(container, child);
         verify(grandchild).close();
-        verify(child).windowClosed();
         verify(container).removeChild(child);
     }