commit 866b88c8d9002847cdf1cccf02c81fc36cfe7653
author enikey <enikey@3789f03b-4d11-0410-bbf8-ca57d06f2519> Tue Jan 20 04:50:55 2009 +0000
committer enikey <enikey@3789f03b-4d11-0410-bbf8-ca57d06f2519> Tue Jan 20 04:50:55 2009 +0000
tree 759c21f4c0b850e9c6e376c51018d3f3bc461594
parent 2f25d35cfa1a4718a01dfe8e73591d8a3e28fe3c

[BugFix] Fixed: Skipping some mouse events when limiting mouse event sending. Now limiting uses thread.

git-svn-id: svn://svn.code.sf.net/p/tigervnc/code/trunk@3561 3789f03b-4d11-0410-bbf8-ca57d06f2519

java/src/com/tightvnc/vncviewer/VncCanvas.java

diff --git a/java/src/com/tightvnc/vncviewer/VncCanvas.java b/java/src/com/tightvnc/vncviewer/VncCanvas.java
index 56019be..4e5460f 100644
--- a/java/src/com/tightvnc/vncviewer/VncCanvas.java
+++ b/java/src/com/tightvnc/vncviewer/VncCanvas.java
@@ -45,7 +45,7 @@
 //
 
 class VncCanvas extends Canvas
-  implements KeyListener, MouseListener, MouseMotionListener, Repaintable {
+  implements KeyListener, MouseListener, MouseMotionListener, Repaintable, Runnable {
 
   VncViewer viewer;
   RfbProto rfb;
@@ -174,6 +174,11 @@
     addKeyListener(this);
     addMouseListener(this);
     addMouseMotionListener(this);
+
+    // Create thread, that will send mouse movement events
+    // to VNC server.
+    Thread mouseThread = new Thread(this);
+    mouseThread.start();
   }
 
   public VncCanvas(VncViewer v) throws IOException {
@@ -771,6 +776,26 @@
     processLocalMouseEvent(evt, true);
   }
 
+  private synchronized void trySendPointerEvent() {
+    if ((needToSendMouseEvent) && (mouseEvent!=null)) {
+      sendMouseEvent(mouseEvent, false);
+      needToSendMouseEvent = false;
+      lastMouseEventSendTime = System.currentTimeMillis();
+    }
+  }
+
+  public void run() {
+    while (true) {
+      // Send mouse movement if we have it
+      trySendPointerEvent();
+      // Sleep for some time
+      try {
+        Thread.sleep(1000 / mouseMaxFreq);
+      } catch (InterruptedException ex) {
+      }
+    }
+  }
+
   //
   // Ignored events.
   //
@@ -817,9 +842,15 @@
     if (viewer.rfb != null && rfb.inNormalProtocol) {
       if (!inSelectionMode) {
         if (inputEnabled) {
-          if (System.currentTimeMillis() - lastMouseEventSendTime >=
-              (1000 / mouseMaxFreq)) {
+          // If mouse not moved, but it's click event then
+          // send it to server immideanlty.
+          // Else, it's mouse movement - we can send it in
+          // our thread later.
+          if (!moved) {
             sendMouseEvent(evt, moved);
+          } else {
+            mouseEvent = evt;
+            needToSendMouseEvent = true;
           }
         }
       } else {
@@ -880,7 +911,8 @@
 
   MemoryImageSource softCursorSource;
   Image softCursor;
-
+  MouseEvent mouseEvent = null;
+  boolean needToSendMouseEvent = false;
   int cursorX = 0, cursorY = 0;
   int cursorWidth, cursorHeight;
   int origCursorWidth, origCursorHeight;