dart-lang / webdev

A CLI for Dart web development.
https://pub.dev/packages/webdev
213 stars 71 forks source link

`yieldControlToDDS` should disconnect non-DDS clients #2399

Open elliette opened 3 months ago

elliette commented 3 months ago

yieldControlToDDS (added here: https://github.com/dart-lang/webdev/pull/1092) currently checks if there is an existing client, and returns false if so.

In fact, DDS should only fail to start if there's an existing DDS instance. In the case of existing clients, the existing clients should be disconnected.

See comment: https://github.com/dart-lang/webdev/pull/2392#discussion_r1539488290