dart2js: Allow 'throw' when inlining

sdk/lib/_internal/compiler/implementation/lib/isolate_helper.dart

Index: sdk/lib/_internal/compiler/implementation/lib/isolate_helper.dart
diff --git a/sdk/lib/_internal/compiler/implementation/lib/isolate_helper.dart b/sdk/lib/_internal/compiler/implementation/lib/isolate_helper.dart
index f47100ec7a8ddfaeb48369ed0fcf5ea516f19d0a..617af04408f2477c946e62c01e1a9565e333bcd8 100644
--- a/sdk/lib/_internal/compiler/implementation/lib/isolate_helper.dart
+++ b/sdk/lib/_internal/compiler/implementation/lib/isolate_helper.dart
@@ -585,7 +585,7 @@ class IsolateNatives {
    * but you should probably not count on this.
    */
   static String _getJSFunctionName(Function f) {
-    return JS("String|Null", r"(#.$name || #)", f, null);
+    return JS("String|Null", r"(#['$name'] || #)", f, null);

[karlklose, 2013/04/09 11:58:46]

Why did you change this?

[erikcorry, 2013/04/10 12:52:32]

When the inliner gets more agressive it can propagate a constant like '1' right
into the JS, so it says ".$name", which is a syntax error.  There are guards to
ensure we never try to execute that, but having a syntax error prevents the
program from even starting.

I am working on using the mini JS parser for these, so that the issue can be
dealt with better.

   }
 
   /** Create a new JavaScript object instance given its constructor. */