From d08ec6ff4a6adb47590d657616fe04d007d8ce66 Mon Sep 17 00:00:00 2001 From: Milian Wolff Date: Wed, 13 Nov 2013 15:28:38 +0100 Subject: Add a test for the property update grouping. For performance reasons, we group property updates and send a single batch update notification. This is now tested to work as intended. This also uncovered a bug in webchannel.js when multiple functions subscribe to the same id. Change-Id: Ic8648d664dd1fe54df7e25fade6a6088386af992 Reviewed-by: Zeno Albisser --- src/webchannel.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src') diff --git a/src/webchannel.js b/src/webchannel.js index 15ae127..7789d84 100644 --- a/src/webchannel.js +++ b/src/webchannel.js @@ -101,7 +101,7 @@ var QWebChannel = function(baseUrl, initCallback) this.subscribe = function(id, callback) { if (channel.subscriptions[id]) { - channel.subscriptions[id].append(callback); + channel.subscriptions[id].push(callback); } else { channel.subscriptions[id] = [callback]; } -- cgit v1.2.1