Address Erik's comments on https://codereview.chromium.org/11778064 and fix minifier failures.

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

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library _isolate_helper;
 
 import 'dart:async';
 import 'dart:isolate';
 
 ReceivePort lazyPort;
@@ skipping 367 matching lines @@
   get id => JS("", "#.id", this);
   void set id(i) { JS("void", "#.id = #", this, i); }
   void set onmessage(f) { JS("void", "#.onmessage = #", this, f); }
   void postMessage(msg) { JS("void", "#.postMessage(#)", this, msg); }
   void terminate() { JS("void", "#.terminate()", this); }
 }
 
 const String _SPAWNED_SIGNAL = "spawned";
 
 var globalThis = IsolateNatives.computeGlobalThis();
-var globalWindow = JS('', "#['window']", globalThis);
-var globalWorker = JS('', "#['Worker']", globalThis);
+var globalWindow = JS('', "#.window", globalThis);
+var globalWorker = JS('', "#.Worker", globalThis);
 bool globalPostMessageDefined =
-    JS('', "#['postMessage'] !== (void 0)", globalThis);
+    JS('', "#.postMessage !== (void 0)", globalThis);
 
 class IsolateNatives {
 
   static String thisScript = computeThisScript();
 
   /**
    * The src url for the script tag that loaded this code. Used to create
    * JavaScript workers.
    */
   static String computeThisScript() {
     // TODO(7369): Find a cross-platform non-brittle way of getting the
     // currently running script.
     var scripts = JS('=List', r"document.getElementsByTagName('script')");
     // The scripts variable only contains the scripts that have already been
     // executed. The last one is the currently running script.
     for (var script in scripts) {
       var src = JS('String|Null', '# && #.src', script, script);
+      // Filter out the test controller script, and the Dart
+      // bootstrap script.
       if (src != null
           && !src.endsWith('test_controller.js')
-          && !new RegExp('client.dart\.js').hasMatch(src)) {

[ngeoffray, 2013/01/16 10:33:38]

The path has now changed. To make things simple, I now only check on 'dart.js'.
Too bad if that's the actual name of the script :-(. Issue 7369 was filed to fix
that.

+          && !src.endsWith('dart.js')) {
         return src;
       }
     }
     return null;
   }
 
   static computeGlobalThis() => JS('', 'function() { return this; }()');
 
   /** Starts a new worker with the given URL. */
   static _WorkerStub _newWorker(url) => JS("_WorkerStub", r"new Worker(#)", url);
@@ skipping 904 matching lines @@
         JS('void', '#.clearInterval(#)', globalThis, _handle);
       }
       _handle = null;
     } else {
       throw new UnsupportedError("Canceling a timer.");
     }
   }
 }
 
 bool hasTimer() => JS('', '#.setTimeout', globalThis) != null;