Do not return a function for property getters on the HTML side.

This simplifies the usage and lets properties be used just like normal
JavaScript properties. This is possible since the properties are cached
on the HTML side.

Change-Id: Ic60076f4596cd8df063567dfbd630e5bd6403119
Reviewed-by: Pierre Rossi <pierre.rossi@gmail.com>

1 files changed, 6 insertions, 17 deletions

diff --git a/src/webchannel/qwebchannel.js b/src/webchannel/qwebchannel.js
index 9529c0b..278f423 100644
--- a/src/webchannel/qwebchannel.js
+++ b/src/webchannel/qwebchannel.js
@@ -389,24 +389,13 @@ function QObject(name, data, webChannel)
 
         });
         object.__defineGetter__(propertyName, function () {
-            return (function (callback) {
-                var propertyValue = object.__propertyCache__[propertyIndex];
-                if (propertyValue === undefined) {
-                    // This shouldn't happen
-                    console.warn("Undefined value in property cache for property \"" + propertyName + "\" in object " + object.__id__);
-                }
+            var propertyValue = object.__propertyCache__[propertyIndex];
+            if (propertyValue === undefined) {
+                // This shouldn't happen
+                console.warn("Undefined value in property cache for property \"" + propertyName + "\" in object " + object.__id__);
+            }
 
-                // TODO: A callback is not required here anymore, but is kept for backwards compatibility
-                if (callback !== undefined) {
-                    if (typeof(callback) !== "function") {
-                        console.error("Bad callback given to get property " + property);
-                        return;
-                    }
-                    callback(propertyValue);
-                } else {
-                    return propertyValue;
-                }
-            });
+            return propertyValue;
         });
     }