Send destroy events for each closed listening socket instead of waiting for all listening sockets t…

Send destroy events for each closed listening socket instead of waiting for all listening sockets to be done

R=sgjesse@google.com

Committed: https://code.google.com/p/dart/source/detail?r=43680

[kustermann, 2015-02-11 11:21:49]

kustermann@google.com changed reviewers:
+ sgjesse@google.com

[kustermann, 2015-02-11 11:21:49]

Missed to add this to the previous CL.

[Søren Gjesse, 2015-02-11 11:31:21]

lgtm

https://codereview.chromium.org/914103002/diff/1/dart/runtime/bin/eventhandle...
File dart/runtime/bin/eventhandler_win.cc (right):

https://codereview.chromium.org/914103002/diff/1/dart/runtime/bin/eventhandle...
dart/runtime/bin/eventhandler_win.cc:1035: DartUtils::PostInt32(msg->dart_port,
1 << kDestroyedEvent);
DeleteIfClosed above sends a destroy event when the last Dart port is closed as
well.

Should this perhapes be in an else?

[kustermann, 2015-02-11 11:32:56]

https://codereview.chromium.org/914103002/diff/1/dart/runtime/bin/eventhandle...
File dart/runtime/bin/eventhandler_win.cc (right):

https://codereview.chromium.org/914103002/diff/1/dart/runtime/bin/eventhandle...
dart/runtime/bin/eventhandler_win.cc:1035: DartUtils::PostInt32(msg->dart_port,
1 << kDestroyedEvent);
On 2015/02/11 11:31:21, Søren Gjesse wrote:
> DeleteIfClosed above sends a destroy event when the last Dart port is closed
as
> well.
> 
> Should this perhapes be in an else?

No it should not. Since we remove the port with `listen_socket->RemovePort()`
DeleteIfClosed() will find no ports left, so it will not send any destroy
events.

[kustermann, 2015-02-11 11:34:55]

Committed patchset #1 (id:1) manually as 43680 (presubmit successful).