flushAsync in string connector is causing multiple messages in single frame

[revelfire]

The spec is unclear on support for this. I am using Spring Websocket support and so far no good with multiple message handling. Working with them - they are considering adding this support but the simplest fix for now is to require one-message-per-frame. This fix does that. The basic problem is if I say:

subscribeJSON(...);
subscribeJSON(...);

say to setup multiple topic listeners in a bootstrapping method, only the first gets registered because they all go in the same frame. This is due to the plugin.dart StringConnector.writeEof() doing flushAsync rather than just flush.

This may be by design for other purposes that I don't understand.

[revelfire]

void writeEof() {
_write(_EOF_STRING);

• _flushAsync();
• _flush();
  +// _flushAsync();
  }

[revelfire]

Let me know if you would prefer a pull request.

[tomyeh]

The fix is correct. It is better to flush immediately right after EOF is written.

I fixed it directly as you suggested. Please do a pull request next time. Thanks.