From f6c2d0c88f76b30ea12cf9576444235ae621ef79 Mon Sep 17 00:00:00 2001
From: Michael Jumper <zhangmaike@users.sourceforge.net>
Date: Sun, 3 Apr 2011 00:53:00 -0700
Subject: [PATCH] Fixed potential layer sync issue (if multiple layers are
 used, and multiple layers are ready, multiply unnecessary sync responses may
 be sent for a single sync instruction)

---
 guacamole-common-js/src/main/resources/guacamole.js | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/guacamole-common-js/src/main/resources/guacamole.js b/guacamole-common-js/src/main/resources/guacamole.js
index 2d2fdce0a..4e4831c36 100644
--- a/guacamole-common-js/src/main/resources/guacamole.js
+++ b/guacamole-common-js/src/main/resources/guacamole.js
@@ -333,17 +333,22 @@ function GuacamoleClient(display, tunnel) {
 
             }
 
-            // Count active layers and install sync tracking hook
+            // Count active, not-ready layers and install sync tracking hooks
             for (var i=0; i<layers.length; i++) {
 
                 var layer = layers[i];
-                if (layer) {
+                if (layer && !layer.isReady()) {
                     layersToSync++;
                     layer.sync(syncLayer);
                 }
 
             }
 
+            // If all layers are ready, then we didn't install any hooks.
+            // Send sync message now,
+            if (layersToSync == 0)
+                tunnel.sendMessage("sync:" + timestamp + ";");
+
         },
       
     };