Move notifyReconnection in AbstractXMPPConnection

smack-bosh/src/main/java/org/jivesoftware/smack/bosh/XMPPBOSHConnection.java

@@ -479,9 +479,7 @@ public class XMPPBOSHConnection extends AbstractXMPPConnection {
                             if (wasAuthenticated) {
                                 login();
                             }
-                            for (ConnectionListener listener : getConnectionListeners()) {
+                            notifyReconnection();
-                                 listener.reconnectionSuccessful();
-                            }
                         }
                         catch (Exception e) {
                             for (ConnectionListener listener : getConnectionListeners()) {

smack-core/src/main/java/org/jivesoftware/smack/AbstractXMPPConnection.java

@@ -1073,6 +1073,23 @@ public abstract class AbstractXMPPConnection implements XMPPConnection {
         }
     }
 
+    /**
+     * Sends a notification indicating that the connection was reconnected successfully.
+     */
+    protected void notifyReconnection() {
+        // Notify connection listeners of the reconnection.
+        for (ConnectionListener listener : getConnectionListeners()) {
+            try {
+                listener.reconnectionSuccessful();
+            }
+            catch (Exception e) {
+                // Catch and print any exception so we can recover
+                // from a faulty listener
+                LOGGER.log(Level.WARNING, "notifyReconnection()", e);
+            }
+        }
+    }
+
     /**
      * A wrapper class to associate a packet filter with a listener.
      */

smack-core/src/test/java/org/jivesoftware/smack/DummyConnection.java

@@ -81,9 +81,7 @@ public class DummyConnection extends AbstractXMPPConnection {
         connectionID = "dummy-" + new Random(new Date().getTime()).nextInt();
 
         if (reconnect) {
-            for (ConnectionListener listener : getConnectionListeners()) {
+            notifyReconnection();
-                listener.reconnectionSuccessful();
-            }
         }
     }
 

smack-tcp/src/main/java/org/jivesoftware/smack/tcp/XMPPTCPConnection.java

@@ -20,7 +20,6 @@ import org.jivesoftware.smack.AbstractXMPPConnection;
 import org.jivesoftware.smack.ConnectionConfiguration;
 import org.jivesoftware.smack.ConnectionConfiguration.SecurityMode;
 import org.jivesoftware.smack.ConnectionCreationListener;
-import org.jivesoftware.smack.ConnectionListener;
 import org.jivesoftware.smack.PacketListener;
 import org.jivesoftware.smack.SmackConfiguration;
 import org.jivesoftware.smack.SmackException;
@@ -843,23 +842,6 @@ public class XMPPTCPConnection extends AbstractXMPPConnection {
         callConnectionClosedOnErrorListener(e);
     }
 
-    /**
-     * Sends a notification indicating that the connection was reconnected successfully.
-     */
-    private void notifyReconnection() {
-        // Notify connection listeners of the reconnection.
-        for (ConnectionListener listener : getConnectionListeners()) {
-            try {
-                listener.reconnectionSuccessful();
-            }
-            catch (Exception e) {
-                // Catch and print any exception so we can recover
-                // from a faulty listener
-                LOGGER.log(Level.WARNING, "notifyReconnection()", e);
-            }
-        }
-    }
-
     /**
      * For unit testing purposes
      *