Stop send mouse clicks while dragging in view only

2 files changed, 9 insertions, 0 deletions

diff --git a/core/rfb.js b/core/rfb.js
index e93be0f..351e4b3 100644
--- a/core/rfb.js
+++ b/core/rfb.js
@@ -836,6 +836,8 @@ export default class RFB extends EventTargetMixin {
                     return;
                 }
 
+                if (this._viewOnly) { return; }
+
                 // Otherwise we treat this as a mouse click event.
                 // Send the button down event here, as the button up
                 // event is sent at the end of this function.
diff --git a/tests/test.rfb.js b/tests/test.rfb.js
index 695e4a3..7c6128e 100644
--- a/tests/test.rfb.js
+++ b/tests/test.rfb.js
@@ -514,6 +514,13 @@ describe('Remote Frame Buffer Protocol Client', function () {
                 expect(RFB.messages.pointerEvent).to.have.been.calledTwice;
             });
 
+            it('should not send button messages when in view only', function () {
+                client._viewOnly = true;
+                client._handleMouseButton(13, 9, 0x001);
+                client._handleMouseButton(13, 9, 0x000);
+                expect(RFB.messages.pointerEvent).to.not.have.been.called;
+            });
+
             it('should send button message directly when drag is disabled', function () {
                 client.dragViewport = false;
                 client._handleMouseButton(13, 9, 0x001);