Start transitioning to new API to start Dart.

client/dart.js

 // 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.
 
 // Bootstrap support for Dart scripts on the page as this script.
 
-if (!document.implementation.hasFeature('dart', '1.0')) {
+if (navigator.webkitStartDart) {
+  navigator.webkitStartDart();
+} else if (!document.implementation.hasFeature('dart', '1.0')) {

[vsm, 2012/02/01 15:35:08]

Are you leaving hasFeature for staging purposes only (it's going away right)?
If so, can you add a TODO to remove that check?

Have you tested loading swarm.html or another sample in both dartium and a
non-dart browser?

[antonm, 2012/02/01 17:07:00]

Yes, hasFeature is only left for staging purposes and I've added a TODO as per
your request.

I haven't tested samples.  I'd prefer to submit and check, so I don't have to
patch URLs (which are currently hardcoded to dartlang.org repo).  I was planning
to do it early my local time to be able to revert immediately if anything fails.

But if you prefer that I check it before, I can do that as well.

On 2012/02/01 15:35:08, vsm wrote:

   window.addEventListener("DOMContentLoaded", function (e) {
       // Fall back to compiled JS.
       var scripts = document.getElementsByTagName("script");
       var length = scripts.length;
       for (var i = 0; i < length; ++i) {
         if (scripts[i].type == "application/dart") {
           // Remap foo.dart to foo.js.
           // TODO:
           // - Support in-browser compilation.
           // - Handle inline Dart scripts.
           if (scripts[i].src && scripts[i].src != '') {
             var script = document.createElement('script');
             script.src = scripts[i].src + '.js';
             var parent = scripts[i].parentNode;
             parent.replaceChild(script, scripts[i]);
           }
         }
       }
     }, false);
 }