Revert "Revert "Dart2js support for custom element lifecycle events""

tools/testing/dart/test_suite.dart

Index: tools/testing/dart/test_suite.dart
diff --git a/tools/testing/dart/test_suite.dart b/tools/testing/dart/test_suite.dart
index 2371140c5b51f52e01270ad5f390506054c72b09..3611976c629fda25824fa1f24f4159118aa4344b 100644
--- a/tools/testing/dart/test_suite.dart
+++ b/tools/testing/dart/test_suite.dart
@@ -1001,7 +1001,22 @@ class StandardTestSuite extends TestSuite {
           dartWrapperFilename = '${htmlPath}_bootstrap.dart';
           compiledDartWrapperFilename = '$dartWrapperFilename.js';
         } else {
-          htmlPath = customHtmlPath;
+          htmlPath = '$tempDir/test.html';
+          dartWrapperFilename = filePath.toNativePath();

[ahe, 2013/09/25 08:37:06]

It's really hard for ricow@ and kustermann@ to keep our testing infrastructure
stable they don't know what is going on. In this particular case, had
kustermann@ been involved in the review, the chances are high that he would have
caught the following problem: this doesn't work with the new way dart2js handles
CSP mode.

The CSP builder turned red, partially due to this CL:
http://build.chromium.org/p/client.dart/builders/dart2js-drt-linux-csp-be/bui...

+
+          var htmlContents = customHtml.readAsStringSync();
+          if (compiler == 'none') {
+            htmlContents = htmlContents.replaceAll('%TEST_SCRIPTS%',
+              '<script type="application/dart" '
+              'src="${_createUrlPathFromFile(filePath)}"></script>\n'
+              '<script type="text/javascript" '
+                  'src="/packages/browser/dart.js"></script>');
+          } else {
+            htmlContents = htmlContents.replaceAll('%TEST_SCRIPTS%',
+              '<script src="$nameNoExt.js"></script>');
+            compiledDartWrapperFilename = '$tempDir/$nameNoExt.js';

[kustermann, 2013/09/25 09:28:27]

Recently dart2js has been changed to emit a "test.precompiled.js" in addition to
"test.js". In csp mode the precompiled version has to be used instead of the
normal one, you can see that below in the else branch.

+          }
+          new File(htmlPath).writeAsStringSync(htmlContents);
         }
       } else {
         htmlPath = '$tempDir/test.html';