Implement try/finally by inlining the finally code.

tests/compiler/dart2js_extra/dart2js_extra.status

Index: tests/compiler/dart2js_extra/dart2js_extra.status
diff --git a/tests/compiler/dart2js_extra/dart2js_extra.status b/tests/compiler/dart2js_extra/dart2js_extra.status
index 5af67cd586aaa4acf4fac829c77e2e6336e3185d..56ed0466d1017991472fb517cfbd55eb13683580 100644
--- a/tests/compiler/dart2js_extra/dart2js_extra.status
+++ b/tests/compiler/dart2js_extra/dart2js_extra.status
@@ -79,8 +79,8 @@ deferred_fail_and_retry_worker_test: SkipByDesign # Uses eval to simulate failed
 22895_test: Crash # (main()async{var clo...  cannot handle async/sync*/async* functions
 23404_test: RuntimeError # Cannot read property 'prototype' of undefined
 LayoutTests_fast_mediastream_getusermedia_t01_test/none: RuntimeError # Cannot read property 'prototype' of undefined
-async_stacktrace_test/asyncStar: Crash # (try {return f(arg1,arg2);}finally {Zone._leave(old);}): try/finally
-async_stacktrace_test/none: Crash # (try {return f(arg1,arg2);}finally {Zone._leave(old);}): try/finally
+async_stacktrace_test/asyncStar: Crash # (runTests()async{awa...  cannot handle async/sync*/async* functions
+async_stacktrace_test/none: Crash # (runTests()async{awa...  cannot handle async/sync*/async* functions
 bounds_check_test/none: RuntimeError # Please triage this failure.
 closure5_test: RuntimeError # Cannot read property 'prototype' of undefined
 closure_capture4_test: RuntimeError # Please triage this failure.
@@ -90,16 +90,16 @@ closure_type_reflection_test: Crash # Internal Error: No default constructor ava
 compound_operator_index_test: RuntimeError # Please triage this failure.
 conditional_send_test: RuntimeError # receiver.get$_collection$_nums is not a function
 conflict_index_test: RuntimeError # Please triage this failure.
-deferred/deferred_class_test: Crash # (try {return f(arg1,arg2);}finally {Zone._leave(old);}): try/finally
-deferred/deferred_constant2_test: Crash # (try {return f(arg1,arg2);}finally {Zone._leave(old);}): try/finally
-deferred/deferred_constant3_test: Crash # (try {return f(arg1,arg2);}finally {Zone._leave(old);}): try/finally
-deferred/deferred_constant4_test: Crash # (try {return f(arg1,arg2);}finally {Zone._leave(old);}): try/finally
-deferred/deferred_function_test: Crash # (try {return f(arg1,arg2);}finally {Zone._leave(old);}): try/finally
-deferred/deferred_mirrors1_test: Crash # (try {return f(arg1,arg2);}finally {Zone._leave(old);}): try/finally
+deferred/deferred_class_test: RuntimeError # receiver.get$_collection$_nums is not a function

[Kevin Millikin (Google), 2015/06/22 15:36:42]

I haven't dug deeply into this, but it seems to be a bug found by increasing
test coverage, not a bug in try/finally itself.

+deferred/deferred_constant2_test: RuntimeError # receiver.get$_collection$_nums is not a function
+deferred/deferred_constant3_test: RuntimeError # receiver.get$_collection$_nums is not a function
+deferred/deferred_constant4_test: RuntimeError # receiver.get$_collection$_nums is not a function
+deferred/deferred_function_test: RuntimeError # receiver.get$_collection$_nums is not a function
+deferred/deferred_mirrors1_test: RuntimeError # receiver.get$_collection$_nums is not a function
 deferred/deferred_mirrors2_test: Crash # Internal Error: No default constructor available.
 deferred/deferred_overlapping_test: RuntimeError # receiver.get$_collection$_nums is not a function
-deferred_fail_and_retry_test: Crash # (try {return f(arg1,arg2);}finally {Zone._leave(old);}): try/finally
-deferred_fail_and_retry_worker_test: Crash # (try {return f(arg1,arg2);}finally {Zone._leave(old);}): try/finally
+deferred_fail_and_retry_test: Crash # Internal Error: No default constructor available.
+deferred_fail_and_retry_worker_test: Crash # Internal Error: No default constructor available.
 if_null_test: RuntimeError # receiver.get$_collection$_nums is not a function
 index_test: RuntimeError # Please triage this failure.
 int_index_test/none: RuntimeError # Please triage this failure.
@@ -112,16 +112,16 @@ mirror_invalid_field_access4_test: RuntimeError # Please triage this failure.
 mirror_invalid_field_access_test: Crash # Internal Error: No default constructor available.
 mirror_invalid_invoke3_test: Crash # Internal Error: No default constructor available.
 mirror_invalid_invoke_test: Crash # Internal Error: No default constructor available.
-mirror_printer_test: Crash # (try {return f(arg1,arg2);}finally {Zone._leave(old);}): try/finally
+mirror_printer_test: Crash # Internal Error: No default constructor available.
 mirror_test: Crash # Internal Error: No default constructor available.
 mirror_type_inference_field2_test: Crash # Internal Error: No default constructor available.
 mirror_type_inference_field_test: Crash # Internal Error: No default constructor available.
 mirror_type_inference_function_test: Crash # Internal Error: No default constructor available.
 mirrors_declarations_filtering_test: Crash # Internal Error: No default constructor available.
 mirrors_used_native_test: RuntimeError # Please triage this failure.
-mirrors_used_warning2_test: Crash # (try {return f(arg);}finally {Zone._leave(old);}): try/finally
-mirrors_used_warning_test/minif: Crash # (try {return f(arg1,arg2);}finally {Zone._leave(old);}): try/finally
-mirrors_used_warning_test/none: Crash # (try {return f(arg1,arg2);}finally {Zone._leave(old);}): try/finally
+mirrors_used_warning2_test: RuntimeError # receiver.get$_nums is not a function
+mirrors_used_warning_test/minif: RuntimeError # receiver.get$_nums is not a function
+mirrors_used_warning_test/none: RuntimeError # receiver.get$_nums is not a function
 no_such_method_mirrors_test: RuntimeError # Please triage this failure.
 reflect_native_types_test: Crash # Internal Error: No default constructor available.
 runtime_type_test: RuntimeError # Cannot read property 'prototype' of undefined