Store end_callback and callback_data into local variables before dropping the monitor

Store end_callback and callback_data into local variables before dropping the monitor

BUG=

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

[Cutch, 2017-01-24 16:01:28]

Description was changed from

==========
Store end_callback and callback_data into local variables before dropping the
monitor

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

to

==========
Store end_callback and callback_data into local variables before dropping the
monitor

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

[Cutch, 2017-01-24 16:01:28]

johnmccutchan@google.com changed reviewers:
+ asiva@google.com

[Cutch, 2017-01-24 16:59:40]

Description was changed from

==========
Store end_callback and callback_data into local variables before dropping the
monitor

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

to

==========
Store end_callback and callback_data into local variables before dropping the
monitor

BUG=

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

[Cutch, 2017-01-24 16:59:41]

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

[rmacnak, 2017-01-24 17:35:09]

rmacnak@google.com changed reviewers:
+ rmacnak@google.com

[rmacnak, 2017-01-24 17:35:10]

lgtm

[siva, 2017-01-24 19:24:26]

lgtm

https://codereview.chromium.org/2656613003/diff/1/runtime/vm/message_handler.cc
File runtime/vm/message_handler.cc (right):

https://codereview.chromium.org/2656613003/diff/1/runtime/vm/message_handler....
runtime/vm/message_handler.cc:438: if (run_end_callback) {
is run_end_callback still needed, it could just be replaced everywhere
with (end_callback != NULL)