Avoid triggering an assert

runtime/vm/kernel_reader.cc

Index: runtime/vm/kernel_reader.cc
diff --git a/runtime/vm/kernel_reader.cc b/runtime/vm/kernel_reader.cc
index 127ad32231e35ef54c487ad1b4f35bdcaf4621c6..e0385c2905f8c8afa6818bfe09bf060a73e582dd 100644
--- a/runtime/vm/kernel_reader.cc
+++ b/runtime/vm/kernel_reader.cc
@@ -213,8 +213,9 @@ Object& KernelReader::ReadProgram() {
       if (procedure != NULL) {
         // We will handle the StaticGet specially and will not use the name.
         //
-        // TODO(kmillikin): we are leaking the function body.  Find a way to
+        // TODO(kmillikin): we are leaking the new function body.  Find a way to
         // deallocate it.
+        delete procedure->function()->body();

[zra, 2017/05/04 20:51:09]

To avoid the hack in set_body, maybe replace these two lines with something
like:

procedure->function()->ReplaceBody(new ReturnStatement(new
StaticGet(NameIndex())));

Maybe you'll still need a hack in ReplaceBody, but it might be better to keep
the hack away from an innocuous-looking setter.

         procedure->function()->set_body(
             new ReturnStatement(new StaticGet(NameIndex())));
       }