Fixed a memory leak when adding too many event listeners to uncoughte…

…xeption in the callbacks for QueuedItem

lib/utils/QueueItem.js

@@ -23,14 +23,21 @@ var QueueItem = module.exports = function(commandName, method, scope, args) {
         nextCallback = function(method, methodName) {
             return function(err,result) {
 
-                process.on('uncaughtException', function (exception) {
+                function errorCB() {
                     self.next();
-                });
+                }
+
+                process.on('uncaughtexception', errorCB);
 
                 // call the callback
                 method.call(scope, err, result);
+
                 // call the next item
                 self.next();
+
+                // remove the listener to avoid memoryleakes. but do it after self.next so we know everything went well
+                process.removeListener('uncaughtexception', errorCB);
+
             };
         };
 
@@ -153,4 +160,4 @@ var QueueItem = module.exports = function(commandName, method, scope, args) {
 
         return child;
     };
-};
+};

package.json

@@ -1,7 +1,7 @@
 {
   "name": "webdriverjs",
   "description": "A nodejs bindings implementation for selenium 2.0/webdriver",
-  "version": "0.7.2",
+  "version": "0.7.3",
   "homepage": "https://github.com/Camme/webdriverjs",
   "author": "camilo tapia <[email protected]>",
   "contributors": [