Update usage of mx_handle_wait_many() API.

Update usage of mx_handle_wait_many() API.

BUG=
R=abarth@google.com

Committed: https://github.com/dart-lang/sdk/commit/46c326fae353c795462ecb63de8d254d49640a38

[jeffbrown, 2016-11-03 04:27:16]

jeffbrown@google.com changed reviewers:
+ abarth@google.com

[jeffbrown, 2016-11-03 04:27:16]

Is there a way to test this?  (Any apps using the dart content handler?)

[abarth, 2016-11-03 04:38:22]

abarth@google.com changed reviewers:
+ zra@google.com

[abarth, 2016-11-03 04:40:00]

On 2016/11/03 at 04:27:16, jeffbrown wrote:
> Is there a way to test this?  (Any apps using the dart content handler?)

You can also try the |dart| command line program on Fuchsia.  That's probably
the easiest way to test it.  zra probably knows how to run the Dart test suite
on Fuchsia.

[jeffbrown, 2016-11-03 05:26:19]

/boot/apps/hello_fuchsia.dart runs, so I'll take it as a good sign.

Ready for review.

[abarth, 2016-11-03 05:29:23]

lgtm

[zra, 2016-11-03 05:48:21]

lgtm

https://codereview.chromium.org/2468423002/diff/20001/runtime/bin/eventhandle...
File runtime/bin/eventhandler_fuchsia.h (right):

https://codereview.chromium.org/2468423002/diff/20001/runtime/bin/eventhandle...
runtime/bin/eventhandler_fuchsia.h:80: mx_handle_t* handles_;
It looks like you can also remove handles_

[abarth-chromium, 2016-11-03 05:48:21]

Description was changed from

==========
Update usage of mx_handle_wait_many() API.

BUG=
==========

to

==========
Update usage of mx_handle_wait_many() API.

BUG=
R=abarth@google.com

Committed:
https://github.com/dart-lang/sdk/commit/46c326fae353c795462ecb63de8d254d49640a38
==========

[abarth-chromium, 2016-11-03 05:48:22]

Committed patchset #2 (id:20001) manually as
46c326fae353c795462ecb63de8d254d49640a38.

[jeffbrown, 2016-11-03 05:56:07]

Thanks for catching that Zach.

Fixed over here: https://codereview.chromium.org/2479453002/

https://codereview.chromium.org/2468423002/diff/20001/runtime/bin/eventhandle...
File runtime/bin/eventhandler_fuchsia.h (right):

https://codereview.chromium.org/2468423002/diff/20001/runtime/bin/eventhandle...
runtime/bin/eventhandler_fuchsia.h:80: mx_handle_t* handles_;
On 2016/11/03 05:48:21, zra wrote:
> It looks like you can also remove handles_

whoops!