From 47043bd0f5a621bf50d0f8e115ecf7e8075ce1ae Mon Sep 17 00:00:00 2001
From: Carlos Ruiz <carg67@gmail.com>
Date: Mon, 27 Mar 2017 18:15:41 +0800
Subject: [PATCH] IDEMPIERE-3653 1007767 Possible memory leak. Fixed memory
 leak. / integrate patch from hengsin

---
 .../adempiere/webui/part/WindowContainer.java  | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

diff --git a/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/part/WindowContainer.java b/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/part/WindowContainer.java
index 0477fd0f16..9e7e06c362 100644
--- a/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/part/WindowContainer.java
+++ b/org.adempiere.ui.zk/WEB-INF/src/org/adempiere/webui/part/WindowContainer.java
@@ -247,8 +247,19 @@ public class WindowContainer extends AbstractUIPart implements EventListener<Eve
      */
     public Tab insertBefore(Tab refTab, Component comp, String title, boolean closeable, boolean enable, DecorateInfo decorateInfo)
     {
-        @SuppressWarnings("serial")
-		final Tab tab = new Tab() {
+        final Menupopup popupClose = new Menupopup();
+        final Tab tab = new Tab() {
+        	/**
+			 * 
+			 */
+			private static final long serialVersionUID = 2387473442130217806L;
+
+			@Override
+        	public void close() {
+        		super.close();
+        		popupClose.detach();
+        	}
+
 			@Override
 			public void onPageDetached(Page page) {
 				super.onPageDetached(page);
@@ -334,8 +345,7 @@ public class WindowContainer extends AbstractUIPart implements EventListener<Eve
         	else
         		setSelectedTab(tab);
         }
-        
-        Menupopup popupClose = new Menupopup();
+
         Menuitem mi;
         if(tab.getIndex()!=0){
         	mi = new Menuitem(Msg.getMsg(Env.getCtx(), "Close"));