fix(VncConsole): disconnect VNC session when unmounting (patternfly#1123

)

The VNC session is properly disconnected when unmounting the VncConsole component. This fixes
memory-leak and asynchronously fired events to an unmounted component.

packages/patternfly-3/react-console/src/VncConsole/VncConsole.js

@@ -18,6 +18,12 @@ const { noop } = helpers;
 class VncConsole extends React.Component {
   state = { status: CONNECTING };
 
+  addEventListeners() {
+    this.rfb.addEventListener('connect', this.onConnected);
+    this.rfb.addEventListener('disconnect', this.onDisconnected);
+    this.rfb.addEventListener('securityfailure', this.onSecurityFailure);
+  }
+
   componentDidMount() {
     const {
       host,
@@ -45,9 +51,7 @@ class VncConsole extends React.Component {
       };
 
       this.rfb = new RFB(this.novncElem, url, options);
-      this.rfb.addEventListener('connect', this.onConnected);
-      this.rfb.addEventListener('disconnect', this.onDisconnected);
-      this.rfb.addEventListener('securityfailure', this.onSecurityFailure);
+      this.addEventListeners();
       this.rfb.viewOnly = viewOnly;
       this.rfb.scaleViewport = false; // if the remote session is smaller than HTML container, the view will be centered
       this.rfb.resizeSession = resizeSession;
@@ -57,6 +61,19 @@ class VncConsole extends React.Component {
     }
   }
 
+  componentWillUnmount() {
+    this.removeEventListeners();
+    this.disconnect();
+  }
+
+  disconnect() {
+    if (!this.rfb) {
+      return;
+    }
+    this.rfb.disconnect();
+    this.rfb = undefined;
+  }
+
   onConnected = () => {
     this.setState({ status: CONNECTED });
   };
@@ -78,6 +95,12 @@ class VncConsole extends React.Component {
     this.props.onSecurityFailure(e);
   };
 
+  removeEventListeners() {
+    this.rfb.removeEventListener('connect', this.onConnected);
+    this.rfb.removeEventListener('disconnect', this.onDisconnected);
+    this.rfb.removeEventListener('securityfailure', this.onSecurityFailure);
+  }
+
   setNovncElem = e => {
     this.novncElem = e;
   };