From: Artur Signell <artur@vaadin.com>
Date: Tue, 10 Apr 2012 12:28:09 +0000 (+0300)
Subject: Disabled excessive debug logging
X-Git-Tag: 7.0.0.alpha2~101
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=66c98850bf12bc9dd6010a2db8907cb2a5ef5e85;p=vaadin-framework.git

Disabled excessive debug logging
---

diff --git a/src/com/vaadin/terminal/gwt/client/ui/AbstractComponentContainerConnector.java b/src/com/vaadin/terminal/gwt/client/ui/AbstractComponentContainerConnector.java
index 2e26a06d1e..7d70729147 100644
--- a/src/com/vaadin/terminal/gwt/client/ui/AbstractComponentContainerConnector.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/AbstractComponentContainerConnector.java
@@ -37,6 +37,8 @@ public abstract class AbstractComponentContainerConnector extends
 
     List<ComponentConnector> children;
 
+    private boolean debugLogging = false;
+
     /**
      * Default constructor
      */
@@ -77,19 +79,21 @@ public abstract class AbstractComponentContainerConnector extends
      * (com.vaadin.terminal.gwt.client.ConnectorHierarchyChangedEvent)
      */
     public void onConnectorHierarchyChange(ConnectorHierarchyChangeEvent event) {
-        // TODO Remove debug info
-        VConsole.log("Hierarchy changed for " + Util.getConnectorString(this));
-        String oldChildren = "* Old children: ";
-        for (ComponentConnector child : event.getOldChildren()) {
-            oldChildren += Util.getConnectorString(child) + " ";
-        }
-        VConsole.log(oldChildren);
+        if (debugLogging) {
+            VConsole.log("Hierarchy changed for "
+                    + Util.getConnectorString(this));
+            String oldChildren = "* Old children: ";
+            for (ComponentConnector child : event.getOldChildren()) {
+                oldChildren += Util.getConnectorString(child) + " ";
+            }
+            VConsole.log(oldChildren);
 
-        String newChildren = "* New children: ";
-        for (ComponentConnector child : getChildren()) {
-            newChildren += Util.getConnectorString(child) + " ";
+            String newChildren = "* New children: ";
+            for (ComponentConnector child : getChildren()) {
+                newChildren += Util.getConnectorString(child) + " ";
+            }
+            VConsole.log(newChildren);
         }
-        VConsole.log(newChildren);
     }
 
     public HandlerRegistration addConnectorHierarchyChangeHandler(
diff --git a/src/com/vaadin/terminal/gwt/client/ui/AbstractConnector.java b/src/com/vaadin/terminal/gwt/client/ui/AbstractConnector.java
index c89a5d0854..9b32399271 100644
--- a/src/com/vaadin/terminal/gwt/client/ui/AbstractConnector.java
+++ b/src/com/vaadin/terminal/gwt/client/ui/AbstractConnector.java
@@ -36,6 +36,7 @@ public abstract class AbstractConnector implements ServerConnector,
 
     private HandlerManager handlerManager;
     private Map<String, Collection<ClientRpc>> rpcImplementations;
+    private final boolean debugLogging = false;
 
     /*
      * (non-Javadoc)
@@ -163,9 +164,11 @@ public abstract class AbstractConnector implements ServerConnector,
     }
 
     public void onStateChanged(StateChangeEvent stateChangeEvent) {
-        VConsole.log("State change event for "
-                + Util.getConnectorString(stateChangeEvent.getConnector())
-                + " received by " + Util.getConnectorString(this));
+        if (debugLogging) {
+            VConsole.log("State change event for "
+                    + Util.getConnectorString(stateChangeEvent.getConnector())
+                    + " received by " + Util.getConnectorString(this));
+        }
     }
 
     /*