Improved AbstractClientConnector performance (#12219)

Avoiding duplicate lookup of connector hierarchy when a component shall be
marked dirty. Optimized if statement to fail fast.

Change-Id: I3ac82d64b235ebfd1888a6076c595f351018d887

diff --git a/server/src/com/vaadin/server/AbstractClientConnector.java b/server/src/com/vaadin/server/AbstractClientConnector.java
index c3101924dd074b74ec024280f94f7ef8ab76b9e0..91a9e41522ed5e2975640cebd8ea50cff086ea0a 100644 (file)
--- a/server/src/com/vaadin/server/AbstractClientConnector.java
+++ b/server/src/com/vaadin/server/AbstractClientConnector.java
@@ -224,9 +224,9 @@ public abstract class AbstractClientConnector implements ClientConnector,
         }
         if (markAsDirty) {
             UI ui = getUI();
-            if (ui != null && !ui.getConnectorTracker().isWritingResponse()
-                    && !ui.getConnectorTracker().isDirty(this)) {
-                markAsDirty();
+            if (ui != null && !ui.getConnectorTracker().isDirty(this)
+                    && !ui.getConnectorTracker().isWritingResponse()) {
+                ui.getConnectorTracker().markDirty(this);
             }
         }
         return sharedState;