Handle empty rects in RAW decoder as well

It was overlooked in the previous commit because we couldn't feed
empty data messages through the test framework.

core/decoders/raw.js

@@ -13,6 +13,10 @@ export default class RawDecoder {
     }
 
     decodeRect(x, y, width, height, sock, display, depth) {
+        if ((width === 0) || (height === 0)) {
+            return true;
+        }
+
         if (this._lines === 0) {
             this._lines = height;
         }

tests/test.copyrect.js

@@ -17,7 +17,13 @@ function testDecodeRect(decoder, x, y, width, height, data, display, depth) {
         decoder.decodeRect(x, y, width, height, sock, display, depth);
     });
 
+    // Empty messages are filtered at multiple layers, so we need to
+    // do a direct call
+    if (data.length === 0) {
+        decoder.decodeRect(x, y, width, height, sock, display, depth);
+    } else {
         sock._websocket._receiveData(new Uint8Array(data));
+    }
 
     display.flip();
 }

tests/test.hextile.js

@@ -17,7 +17,13 @@ function testDecodeRect(decoder, x, y, width, height, data, display, depth) {
         decoder.decodeRect(x, y, width, height, sock, display, depth);
     });
 
+    // Empty messages are filtered at multiple layers, so we need to
+    // do a direct call
+    if (data.length === 0) {
+        decoder.decodeRect(x, y, width, height, sock, display, depth);
+    } else {
         sock._websocket._receiveData(new Uint8Array(data));
+    }
 
     display.flip();
 }

tests/test.raw.js

@@ -17,7 +17,13 @@ function testDecodeRect(decoder, x, y, width, height, data, display, depth) {
         decoder.decodeRect(x, y, width, height, sock, display, depth);
     });
 
+    // Empty messages are filtered at multiple layers, so we need to
+    // do a direct call
+    if (data.length === 0) {
+        decoder.decodeRect(x, y, width, height, sock, display, depth);
+    } else {
         sock._websocket._receiveData(new Uint8Array(data));
+    }
 
     display.flip();
 }

tests/test.rre.js

@@ -17,7 +17,13 @@ function testDecodeRect(decoder, x, y, width, height, data, display, depth) {
         decoder.decodeRect(x, y, width, height, sock, display, depth);
     });
 
+    // Empty messages are filtered at multiple layers, so we need to
+    // do a direct call
+    if (data.length === 0) {
+        decoder.decodeRect(x, y, width, height, sock, display, depth);
+    } else {
         sock._websocket._receiveData(new Uint8Array(data));
+    }
 
     display.flip();
 }

tests/test.tight.js

@@ -17,7 +17,13 @@ function testDecodeRect(decoder, x, y, width, height, data, display, depth) {
         decoder.decodeRect(x, y, width, height, sock, display, depth);
     });
 
+    // Empty messages are filtered at multiple layers, so we need to
+    // do a direct call
+    if (data.length === 0) {
+        decoder.decodeRect(x, y, width, height, sock, display, depth);
+    } else {
         sock._websocket._receiveData(new Uint8Array(data));
+    }
 
     display.flip();
 }

tests/test.tightpng.js

@@ -17,7 +17,13 @@ function testDecodeRect(decoder, x, y, width, height, data, display, depth) {
         decoder.decodeRect(x, y, width, height, sock, display, depth);
     });
 
+    // Empty messages are filtered at multiple layers, so we need to
+    // do a direct call
+    if (data.length === 0) {
+        decoder.decodeRect(x, y, width, height, sock, display, depth);
+    } else {
         sock._websocket._receiveData(new Uint8Array(data));
+    }
 
     display.flip();
 }