Merge pull request #1 from binary-hamedanchi/mohammad/ie_memory_leak

mohammad/ie_memory_leak

Author: aminroosta

src/javascript/binary/websocket_pages/trade/beta/tradepage.js

@@ -62,6 +62,7 @@ var TradePage_Beta = (function(){
     BinarySocket.clear();
     Defaults.clear();
     PortfolioWS.onUnload();
+    chartFrameCleanup();
   };
 
   return {

src/javascript/binary/websocket_pages/trade/common.js

@@ -911,19 +911,24 @@ function showHighchart(){
   if (window.chartAllowed) {
     chartFrameSource();
   } else {
-    /*
-     * Prevent IE memory leak (http://stackoverflow.com/questions/8407946).
-     */
-    document.getElementById('chart_frame').src = 'about:blank';
+    chartFrameCleanup();
     $('#trade_live_chart').hide();
     $('#chart-error').text(page.text.localize('Chart is not available for this underlying.'))
                      .show();
     return;
   }
 }
 
+function chartFrameCleanup() {
+    /*
+     * Prevent IE memory leak (http://stackoverflow.com/questions/8407946).
+     */
+    document.getElementById('chart_frame').src = 'about:blank';
+}
+
 function chartFrameSource() {
-  if ($('#tab_graph').hasClass('active') && (sessionStorage.getItem('old_underlying') !== sessionStorage.getItem('underlying') || $('#chart_frame').attr('src') === '' || $('#chart_frame').attr('src') === 'about:blank')) {
+  if ($('#tab_graph').hasClass('active') && (sessionStorage.getItem('old_underlying') !== sessionStorage.getItem('underlying') || /^(|about:blank)$/.test($('#chart_frame').attr('src')))) {
+      chartFrameCleanup();
       setChartSource();
       sessionStorage.setItem('old_underlying', document.getElementById('underlying').value);
   }
@@ -1163,6 +1168,7 @@ module.exports = {
     reloadPage: reloadPage,
     addComma: addComma,
     showHighchart: showHighchart,
+    chartFrameCleanup: chartFrameCleanup,
     chartFrameSource: chartFrameSource,
     displayContractForms: displayContractForms,
     displayMarkets: displayMarkets,

src/javascript/binary/websocket_pages/trade/tradepage.js

@@ -60,6 +60,7 @@ var TradePage = (function(){
     forgetTradingStreams();
     BinarySocket.clear();
     Defaults.clear();
+    chartFrameCleanup();
   };
 
   return {

src/javascript/binary_japan/trade_japan/JPTradePage.js

@@ -36,6 +36,7 @@ var JPTradePage = (function() {
   };
 
   var onUnload = function() {
+    chartFrameCleanup();
     window.chartAllowed = false;
     JapanPortfolio.hide();
     isJapan = false;