retVal variable pollutes global scope

[vsviridov]

I'm not sure if this is by design, but there's a bunch of places with retVal variable is not prefixed with var, so it ends up in global scope.

Cross-posted from Socket.IO's tracker, since it uses web-socket-js as a shim.

[gimite]

I guess it's simply a bug. FABridge is third-party library, so I'm not exactly sure, though.

[gimite]

Note that FABridge is external library, so I'd like to keep the change minimum. e.g. I want to keep the original code style, and will fix only obvious bugs such as dropping "var".

[gimite]

This bug is now obsolete because we stopped using FABridge.