Author: adietish
Date: 2010-12-09 07:27:26 -0500 (Thu, 09 Dec 2010)
New Revision: 27274
Modified:
trunk/deltacloud/plugins/org.jboss.tools.deltacloud.ui/ChangeLog
trunk/deltacloud/plugins/org.jboss.tools.deltacloud.ui/src/org/jboss/tools/deltacloud/ui/views/cloudelements/AbstractCloudElementTableView.java
Log:
[JBIDE-7853] removing workbench selection listener on dispose
Modified: trunk/deltacloud/plugins/org.jboss.tools.deltacloud.ui/ChangeLog
===================================================================
--- trunk/deltacloud/plugins/org.jboss.tools.deltacloud.ui/ChangeLog 2010-12-09 12:19:57
UTC (rev 27273)
+++ trunk/deltacloud/plugins/org.jboss.tools.deltacloud.ui/ChangeLog 2010-12-09 12:27:26
UTC (rev 27274)
@@ -2,6 +2,8 @@
*
src/org/jboss/tools/deltacloud/ui/views/cloudelements/AbstractCloudElementTableView.java
(.selectionChanged):
[JBIDE-7849] only change selected cloud if a new cloud has been selected
+ *
src/org/jboss/tools/deltacloud/ui/views/cloudelements/AbstractCloudElementTableView.java
(dispose):
+ [JBIDE-7853] removing workbench selection listener on dispose
2010-12-08 André Dietisheim <adietish(a)redhat.com>
Modified:
trunk/deltacloud/plugins/org.jboss.tools.deltacloud.ui/src/org/jboss/tools/deltacloud/ui/views/cloudelements/AbstractCloudElementTableView.java
===================================================================
---
trunk/deltacloud/plugins/org.jboss.tools.deltacloud.ui/src/org/jboss/tools/deltacloud/ui/views/cloudelements/AbstractCloudElementTableView.java 2010-12-09
12:19:57 UTC (rev 27273)
+++
trunk/deltacloud/plugins/org.jboss.tools.deltacloud.ui/src/org/jboss/tools/deltacloud/ui/views/cloudelements/AbstractCloudElementTableView.java 2010-12-09
12:27:26 UTC (rev 27274)
@@ -150,12 +150,6 @@
protected abstract String getSelectedCloudPrefsKey();
@Override
- public void dispose() {
- DeltaCloudManager.getDefault().removeCloudManagerListener(this);
- super.dispose();
- }
-
- @Override
public void createPartControl(Composite parent) {
Composite container = new Composite(parent, SWT.NULL);
FormLayout layout = new FormLayout();
@@ -395,4 +389,12 @@
public void setFocus() {
viewer.getControl().setFocus();
}
+
+ @Override
+ public void dispose() {
+ getSite().getWorkbenchWindow().getSelectionService().removePostSelectionListener(workbenchSelectionListener);
+ DeltaCloudManager.getDefault().removeCloudManagerListener(this);
+ super.dispose();
+ }
+
}
Show replies by date