183087 - Windows tree's don't have to always have scroll bars

1 files changed, 20 insertions, 2 deletions

diff --git a/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/widgets/ScrollBar.java b/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/widgets/ScrollBar.java
index a4b5f17c7a..704d9cf811 100644
--- a/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/widgets/ScrollBar.java
+++ b/bundles/org.eclipse.swt/Eclipse SWT/gtk/org/eclipse/swt/widgets/ScrollBar.java
@@ -146,6 +146,16 @@ void deregister () {
 	if (adjustmentHandle != 0) display.removeWidget (adjustmentHandle);
 }
 
+void destroyHandle () {
+	super.destroyWidget ();
+}
+
+void destroyWidget () {
+	parent.destroyScrollBar (this);
+	releaseHandle ();
+	//parent.sendEvent (SWT.Resize);
+}
+
 /**
  * Returns <code>true</code> if the receiver is enabled, and
  * <code>false</code> otherwise. A disabled control is typically
@@ -459,6 +469,11 @@ void register () {
 	if (adjustmentHandle != 0) display.addWidget (adjustmentHandle, this);
 }
 
+void releaseHandle () {
+	super.releaseHandle ();
+	parent = null;
+}
+
 void releaseParent () {
 	super.releaseParent ();
 	if (parent.horizontalBar == this) parent.horizontalBar = null;
@@ -467,7 +482,7 @@ void releaseParent () {
 
 void releaseWidget () {
 	super.releaseWidget ();
-	parent = null;
+	//parent = null;
 }
 
 /**
@@ -741,7 +756,10 @@ public void setValues (int selection, int minimum, int maximum, int thumb, int i
  */
 public void setVisible (boolean visible) {
 	checkWidget ();
-	parent.setScrollBarVisible (this, visible);
+	if (parent.setScrollBarVisible (this, visible)) {
+		sendEvent (visible ? SWT.Show : SWT.Hide);
+		parent.sendEvent (SWT.Resize);
+	}
 }
 
 }