Properly remove shadow event listeners to prevent IE8 memory leak (#13129)

Change-Id: I98becf08f56ce35a3ee47650d0d4e204e7d18e73

diff --git a/client/src/com/vaadin/client/ui/VOverlay.java b/client/src/com/vaadin/client/ui/VOverlay.java
index 2f5df6d4f36afbad5c891131c2efd6d341a84c85..e2c9001fed0e86ea27a39210faa4d7bb4a4165e1 100644 (file)
--- a/client/src/com/vaadin/client/ui/VOverlay.java
+++ b/client/src/com/vaadin/client/ui/VOverlay.java
@@ -241,10 +241,10 @@ public class VOverlay extends PopupPanel implements CloseHandler<PopupPanel> {
 
     private void removeShadowIfPresent() {
         if (isShadowAttached()) {
-            shadow.removeFromParent();
-
             // Remove event listener from the shadow
             unsinkShadowEvents();
+
+            shadow.removeFromParent();
         }
     }