Removing polymer-specific code from barback runner

pkg/polymer/lib/deploy.dart

 // Copyright (c) 2013, 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.
 
 /**
  * **Note**: If you already have a `build.dart` in your application, we
  * recommend to use the `package:polymer/builder.dart` library instead.
 
  * Temporary deploy command used to create a version of the app that can be
  * compiled with dart2js and deployed. Following pub layout conventions, this
@@ skipping 26 matching lines @@
 
   var test = args['test'];
   var outDir = args['out'];
 
   var options;
   if (test == null) {
     var transformOps = new TransformOptions(
         directlyIncludeJS: args['js'],
         contentSecurityPolicy: args['csp'],
         releaseMode: !args['debug']);
-    options = new BarbackOptions(createDeployPhases(transformOps), outDir);
+    var phases = createDeployPhases(transformOps);
+    options = new BarbackOptions(phases, outDir,

[Siggi Cherem (dart-lang), 2014/02/11 23:15:20]

it's fine in it's current form, but I wonder if we want to make phases here be
also a packagePhase with the key being the current package.

[blois, 2014/02/12 02:39:27]

Yeah- I was considering removing the phases completely, but it seems like it's
the most common- to run the transform on the current package. And since the
current package can be auto-detected it seems lame to have to specify it.

[Siggi Cherem (dart-lang), 2014/02/12 03:05:28]

sounds good

+        packagePhases: {'polymer': phases});
   } else {
     options = _createTestOptions(
         test, outDir, args['js'], args['csp'], !args['debug']);
   }
   if (options == null) exit(1);
 
   print('polymer/deploy.dart: creating a deploy target for '
       '"${options.currentPackage}"');
 
   runBarback(options)
@@ skipping 19 matching lines @@
   var packageName = readCurrentPackageFromPubspec(pubspecDir);
 
   var phases = createDeployPhases(new TransformOptions(
       entryPoints: [path.relative(testFile, from: pubspecDir)],
       directlyIncludeJS: directlyIncludeJS,
       contentSecurityPolicy: contentSecurityPolicy,
       releaseMode: releaseMode));
   return new BarbackOptions(phases, outDir,
       currentPackage: packageName,
       packageDirs: {packageName : pubspecDir},
+      packagePhases: {'polymer': phases},
       transformTests: true);
 }
 
 String _findDirWithFile(String dir, String filename) {
   while (!new File(path.join(dir, filename)).existsSync()) {
     var parentDir = path.dirname(dir);
     // If we reached root and failed to find it, bail.
     if (parentDir == dir) return null;
     dir = parentDir;
   }
@@ skipping 39 matching lines @@
     _showUsage(parser);
     return null;
   }
 }
 
 _showUsage(parser) {
   print('Usage: dart --package-root=packages/ '
       'package:polymer/deploy.dart [options]');
   print(parser.getUsage());
 }