From: Jouni Koivuviita <jouni.koivuviita@itmill.com>
Date: Thu, 3 Apr 2008 09:47:26 +0000 (+0000)
Subject: Fix: Force browser to fire unload event when component is detached from the view... 
X-Git-Tag: 6.7.0.beta1~4917
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=95d4f7757851fcdc1ff4ded1033f254e892eca92;p=vaadin-framework.git

Fix: Force browser to fire unload event when component is detached from the view (IE doesn't do this automatically).

svn changeset:4126/svn branch:trunk
---

diff --git a/src/com/itmill/toolkit/terminal/gwt/client/ui/IEmbedded.java b/src/com/itmill/toolkit/terminal/gwt/client/ui/IEmbedded.java
index 50b3c3cfe4..7c27117944 100644
--- a/src/com/itmill/toolkit/terminal/gwt/client/ui/IEmbedded.java
+++ b/src/com/itmill/toolkit/terminal/gwt/client/ui/IEmbedded.java
@@ -73,7 +73,7 @@ public class IEmbedded extends HTML implements Paintable {
      * @return
      */
     private String getSrc(UIDL uidl, ApplicationConnection client) {
-        String url =  client.translateToolkitUri(uidl.getStringAttribute("src"));
+        String url = client.translateToolkitUri(uidl.getStringAttribute("src"));
         if (url == null) {
             return "";
         }
@@ -95,4 +95,13 @@ public class IEmbedded extends HTML implements Paintable {
         heigth = height;
         super.setHeight(height);
     }
+
+    protected void onDetach() {
+        // Force browser to fire unload event when component is detached from
+        // the view (IE doesn't do this automatically)
+        if (browserElement != null) {
+            DOM.setElementAttribute(browserElement, "src", "javascript:false");
+        }
+        super.onDetach();
+    }
 }