Tidy up some windowClosed/windowClosing methods.

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

src/com/dmdirc/addons/dcc/DCCFrameContainer.java

@@ -102,10 +102,4 @@ public abstract class DCCFrameContainer extends WritableFrameContainer {
         // 5: Inform any parents that the window is closing
     }
 
-    /** {@inheritDoc} */
-    @Override
-    public void windowClosed() {
-        // 7: Remove any references to the window and parents
-    }
-
 }

src/com/dmdirc/addons/dcc/PlaceholderContainer.java

@@ -116,12 +116,6 @@ public class PlaceholderContainer extends FrameContainer {
         plugin.removeContainer();
     }
 
-    /** {@inheritDoc} */
-    @Override
-    public void windowClosed() {
-        // 7: Remove any references to the window and parents
-    }
-
     /** {@inheritDoc} */
     @Override
     public void removeChild(final FrameContainer child) {

src/com/dmdirc/addons/dcc/TransferContainer.java

@@ -364,12 +364,6 @@ public class TransferContainer extends FrameContainer implements
         // 5: Inform any parents that the window is closing
     }
 
-    /** {@inheritDoc} */
-    @Override
-    public void windowClosed() {
-        // 7: Remove any references to the window and parents
-    }
-
     public void addSocketCloseCallback(final SocketCloseListener listener) {
         if (connection != null && connection.getParser() != null) {
             connection.getParser().getCallbackManager()

src/com/dmdirc/addons/logging/HistoryWindow.java

@@ -52,21 +52,6 @@ public class HistoryWindow extends FrameContainer {
         addLine(reader.getLinesAsString(Math.min(frameBufferSize, numLines)), false);
     }
 
-    /** {@inheritDoc} */
-    @Override
-    public void windowClosing() {
-        // 2: Remove any callbacks or listeners
-        // 3: Trigger any actions neccessary
-        // 4: Trigger action for the window closing
-        // 5: Inform any parents that the window is closing
-    }
-
-    /** {@inheritDoc} */
-    @Override
-    public void windowClosed() {
-        // 7: Remove any references to the window and parents
-    }
-
     /** {@inheritDoc} */
     @Override
     public Connection getConnection() {

src/com/dmdirc/addons/parserdebug/DebugWindow.java

@@ -37,11 +37,11 @@ import java.util.Arrays;
 public class DebugWindow extends FrameContainer {
 
     /** The plugin that owns this window */
-    protected DebugPlugin plugin;
+    protected final DebugPlugin plugin;
     /** The parser this window is debugging */
     protected Parser parser;
     /** The connection we're operating on */
-    protected Connection connection;
+    protected final Connection connection;
 
     /**
      * Creates a new instance of DebugWindow.
@@ -79,9 +79,8 @@ public class DebugWindow extends FrameContainer {
         parser = null;
     }
 
-
     /**
-     * Closes this container (and it's associated frame).
+     * Closes this container (and its associated frame).
      */
     @Override
     public void windowClosing() {
@@ -95,12 +94,4 @@ public class DebugWindow extends FrameContainer {
         // 5: Inform any parents that the window is closing
     }
 
-    /** {@inheritDoc} */
-    @Override
-    public void windowClosed() {
-        // 7: Remove any references to the window and parents
-        this.parser = null;
-        this.connection = null;
-        this.plugin = null;
-    }
 }

src/com/dmdirc/addons/redirect/FakeWriteableFrameContainer.java

@@ -116,15 +116,4 @@ public class FakeWriteableFrameContainer extends WritableFrameContainer {
         return target.getConnection();
     }
 
-    /** {@inheritDoc} */
-    @Override
-    public void windowClosing() {
-        // Do nothing
-    }
-
-    /** {@inheritDoc} */
-    @Override
-    public void windowClosed() {
-        // Do nothing
-    }
 }

src/com/dmdirc/addons/swingdebug/SystemStreamContainer.java

@@ -77,10 +77,4 @@ public class SystemStreamContainer extends FrameContainer {
         // 5: Inform any parents that the window is closing
     }
 
-    /** {@inheritDoc} */
-    @Override
-    public void windowClosed() {
-        // 7: Remove any references to the window and parents
-    }
-
 }