From de98e505a25b2cae5e2152d540074d60c7bc530a Mon Sep 17 00:00:00 2001
From: Artur Signell <artur.signell@itmill.com>
Date: Tue, 16 Mar 2010 12:34:44 +0000
Subject: [PATCH] Fix for #4354 - SplitPanel setComponent methods should check
 if the component is already set

svn changeset:11908/svn branch:6.3
---
 src/com/vaadin/ui/SplitPanel.java | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/src/com/vaadin/ui/SplitPanel.java b/src/com/vaadin/ui/SplitPanel.java
index 1a149d7394..4319bf7d9c 100644
--- a/src/com/vaadin/ui/SplitPanel.java
+++ b/src/com/vaadin/ui/SplitPanel.java
@@ -105,6 +105,11 @@ public class SplitPanel extends AbstractLayout {
     }
 
     public void setFirstComponent(Component c) {
+        if (firstComponent == c) {
+            // Nothing to do
+            return;
+        }
+
         if (firstComponent != null) {
             // detach old
             removeComponent(firstComponent);
@@ -114,6 +119,11 @@ public class SplitPanel extends AbstractLayout {
     }
 
     public void setSecondComponent(Component c) {
+        if (c == secondComponent) {
+            // Nothing to do
+            return;
+        }
+
         if (secondComponent != null) {
             // detach old
             removeComponent(secondComponent);
-- 
2.39.5