From 9aa35f067cdc532faa8cbac250dda2419d577a12 Mon Sep 17 00:00:00 2001
From: Henri Sara <henri.sara@itmill.com>
Date: Thu, 1 Oct 2009 12:37:56 +0000
Subject: [PATCH] Merged [9008] from 6.1 to 6.2: #3381 Detached
 UriFragmentUtility should not send events

svn changeset:9009/svn branch:6.2
---
 .../gwt/client/ui/VUriFragmentUtility.java      | 17 +++++++++++++++--
 1 file changed, 15 insertions(+), 2 deletions(-)

diff --git a/src/com/vaadin/terminal/gwt/client/ui/VUriFragmentUtility.java b/src/com/vaadin/terminal/gwt/client/ui/VUriFragmentUtility.java
index c67de9aebf..f0d571e741 100644
--- a/src/com/vaadin/terminal/gwt/client/ui/VUriFragmentUtility.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/VUriFragmentUtility.java
@@ -3,6 +3,7 @@ package com.vaadin.terminal.gwt.client.ui;
 import com.google.gwt.dom.client.Document;
 import com.google.gwt.event.logical.shared.ValueChangeEvent;
 import com.google.gwt.event.logical.shared.ValueChangeHandler;
+import com.google.gwt.event.shared.HandlerRegistration;
 import com.google.gwt.user.client.History;
 import com.google.gwt.user.client.ui.Widget;
 import com.vaadin.terminal.gwt.client.ApplicationConnection;
@@ -13,7 +14,7 @@ import com.vaadin.terminal.gwt.client.UIDL;
 /**
  * Client side implementation for UriFragmentUtility. Uses GWT's History object
  * as an implementation.
- * 
+ *
  */
 public class VUriFragmentUtility extends Widget implements Paintable,
         ValueChangeHandler<String> {
@@ -22,6 +23,7 @@ public class VUriFragmentUtility extends Widget implements Paintable,
     private ApplicationConnection client;
     private String paintableId;
     private boolean immediate;
+    private HandlerRegistration historyValueHandlerRegistration;
 
     public VUriFragmentUtility() {
         setElement(Document.get().createDivElement());
@@ -29,10 +31,21 @@ public class VUriFragmentUtility extends Widget implements Paintable,
             getElement().getStyle().setProperty("overflow", "hidden");
             getElement().getStyle().setProperty("height", "0");
         }
-        History.addValueChangeHandler(this);
+    }
+
+    @Override
+    protected void onAttach() {
+        super.onAttach();
+        historyValueHandlerRegistration = History.addValueChangeHandler(this);
         History.fireCurrentHistoryState();
     }
 
+    @Override
+    protected void onDetach() {
+        super.onDetach();
+        historyValueHandlerRegistration.removeHandler();
+    }
+
     public void updateFromUIDL(UIDL uidl, ApplicationConnection client) {
         if (client.updateComponent(this, uidl, false)) {
             return;
-- 
2.39.5